Project

General

Profile

Actions

Bug #14724

closed

ceph-disk: list test fail

Added by Loïc Dachary about 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description


    ============================= test session starts ==============================
    platform linux2 -- Python 2.7.9, pytest-2.8.7, py-1.4.31, pluggy-0.3.1 -- /nvm/src/ceph2/src/ceph-disk/.tox/py27/bin/python2.7
    cachedir: .cache
    rootdir: /nvm/src/ceph2/src/ceph-disk, inifile:
    collecting ... collected 25 items

    tests/test_main.py::TestCephDisk::test_main_list_json PASSED
    tests/test_main.py::TestCephDisk::test_main_list_plain PASSED
    tests/test_main.py::TestCephDisk::test_list_format_more_osd_info_plain PASSED
    tests/test_main.py::TestCephDisk::test_list_format_plain PASSED
    tests/test_main.py::TestCephDisk::test_list_format_dev_plain PASSED
    tests/test_main.py::TestCephDisk::test_list_dev_osd PASSED
    tests/test_main.py::TestCephDisk::test_list_all_partitions PASSED
    tests/test_main.py::TestCephDisk::test_list_data PASSED
    tests/test_main.py::TestCephDisk::test_list_dmcrypt_data FAILED
    tests/test_main.py::TestCephDisk::test_list_multipath PASSED
    tests/test_main.py::TestCephDisk::test_list_default PASSED
    tests/test_main.py::TestCephDisk::test_list_bluestore PASSED
    tests/test_main.py::TestCephDisk::test_list_other PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_check_osd_status PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_deallocate_osd_id_fail PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_delete_osd_auth_key_fail PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_main_deactivate PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_main_destroy PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_mark_out_out PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_mount PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_path_set_context PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_remove_from_crush_map_fail PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_remove_osd_directory_files PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_stop_daemon PASSED
    tests/test_main.py::TestCephDiskDeactivateAndDestroy::test_umount PASSED

    =================================== FAILURES ===================================
    _____________________ TestCephDisk.test_list_dmcrypt_data ______________________

    self = <test_main.TestCephDisk object at 0x7fd7ccb6b8d0>

        def test_list_dmcrypt_data(self):
            partition_type2type = {
                main.PTYPE['plain']['osd']['ready']: 'plain',
                main.PTYPE['luks']['osd']['ready']: 'LUKS',
            }
            for (partition_type, type) in partition_type2type.iteritems():
                #
                # dmcrypt data partition with one holder
                #
                partition_uuid = "56244cf5-83ef-4984-888a-2d8b8e0e04b2" 
                disk = "Xda" 
                partition = "Xda1" 
                holders = ["dm-0"]
                with patch.multiple(
                        main,
                        is_held=lambda dev: holders,
                        list_all_partitions=lambda: {disk: [partition]},
                        get_partition_uuid=lambda dev: partition_uuid,
                        get_partition_type=lambda dev: partition_type,
                        is_partition=lambda dev: True,
                ):
                    expect = [{'path': '/dev/' + disk,
                               'partitions': [{
                                   'dmcrypt': {
                                       'holders': holders,
                                       'type': type,
                                   },
                                   'fs_type': None,
                                   'is_partition': True,
                                   'mount': None,
                                   'path': '/dev/' + partition,
                                   'ptype': partition_type,
                                   'state': 'unprepared',
                                   'type': 'data',
                                   'uuid': partition_uuid,
                               }]}]
    >               assert expect == main.list_devices()
    E               assert [{'partitions...: '/dev/Xda'}] == [{'partitions'...: '/dev/Xda'}]
    E                 At index 0 diff: {'path': '/dev/Xda', 'partitions': [{'fs_type': None, 'ptype': '4fbd7e29-9d25-41b8-afd0-5ec00ceff05d', 'is_partition': True, 'type': 'data', 'uuid': '56244cf5-83ef-4984-888a-2d8b8e0e04b2', 'mount': None, 'path': '/dev/Xda1', 'state': 'unprepared', 'dmcrypt': {'holders': ['dm-0'], 'type': 'plain'}}]} != {'path': '/dev/Xda', 'partitions': [{'fs_type': None, 'ceph_fsid': None, 'whoami': None, 'is_partition': True, 'type': 'data', 'path': '/dev/Xda1', 'state': 'active', 'ptype': '4fbd7e29-9d25-41b8-afd0-5ec00ceff05d', 'uuid': '56244cf5-83ef-4984-888a-2d8b8e0e04b2', 'mount': '/', 'dmcrypt': {'holders': ['dm-0'], 'type': 'plain'}}]}
    E                 Full diff:
    E                 - [{'partitions': [{'dmcrypt': {'holders': ['dm-0'], 'type': 'plain'},
    E                 + [{'partitions': [{'ceph_fsid': None,
    E                 +                   'dmcrypt': {'holders': ['dm-0'], 'type': 'plain'},
    E                 'fs_type': None,
    E                 'is_partition': True,
    E                 -                   'mount': None,
    E                 ?                            ^^^^
    E                 +                   'mount': '/',
    E                 ?                            ^^^
    E                 'path': '/dev/Xda1',
    E                 'ptype': '4fbd7e29-9d25-41b8-afd0-5ec00ceff05d',
    E                 -                   'state': 'unprepared',
    E                 ?                             ^^^^ -----
    E                 +                   'state': 'active',
    E                 ?                             ^^^^^
    E                 'type': 'data',
    E                 -                   'uuid': '56244cf5-83ef-4984-888a-2d8b8e0e04b2'}],
    E                 ?                                                                 --
    E                 +                   'uuid': '56244cf5-83ef-4984-888a-2d8b8e0e04b2',
    E                 +                   'whoami': None}],
    E                 'path': '/dev/Xda'}]

    tests/test_main.py:342: AssertionError
    ----------------------------- Captured stderr call -----------------------------
    DEBUG:ceph_disk.main:main_list: /dev/Xda1 ptype = 4fbd7e29-9d25-41b8-afd0-5ec00ceff05d uuid = 56244cf5-83ef-4984-888a-2d8b8e0e04b2
    INFO:ceph_disk.main:Running command: /usr/sbin/blkid -s TYPE /dev/dm-0
    DEBUG:ceph_disk.main:main_list: {'Xda': ['Xda1']}, uuid_map = {'56244cf5-83ef-4984-888a-2d8b8e0e04b2': '/dev/Xda1'}, space_map = {}
    INFO:ceph_disk.main:list_dev(dev = /dev/Xda1, ptype = 4fbd7e29-9d25-41b8-afd0-5ec00ceff05d)
    INFO:ceph_disk.main:Running command: /usr/sbin/blkid -s TYPE /dev/dm-0
    DEBUG:ceph_disk.main:list_devices: [{'path': '/dev/Xda', 'partitions': [{'fs_type': None, 'ceph_fsid': None, 'whoami': None, 'is_partition': True, 'type': 'data', 'path': '/dev/Xda1', 'state': 'active', 'ptype': '4fbd7e29-9d25-41b8-afd0-5ec00ceff05d', 'uuid': '56244cf5-83ef-4984-888a-2d8b8e0e04b2', 'mount': '/', 'dmcrypt': {'holders': ['dm-0'], 'type': 'plain'}}]}]
    ===================== 1 failed, 24 passed in 5.00 seconds ======================

Actions #1

Updated by Loïc Dachary about 8 years ago

  • Status changed from 12 to Resolved
Actions

Also available in: Atom PDF