]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
scrub_test: Fixes need with wip-13507
authorDavid Zafman <dzafman@redhat.com>
Fri, 29 Apr 2016 22:33:44 +0000 (15:33 -0700)
committerDavid Zafman <dzafman@redhat.com>
Tue, 3 May 2016 16:11:33 +0000 (09:11 -0700)
Accept an attribute if not present as False
Change required due change to dict with "epoch" and "inconsistents"

Signed-off-by: David Zafman <dzafman@redhat.com>
tasks/scrub_test.py

index a2ab2c46c69d8a27afccd17d99efe7e74d337ea2..23ab8b954b1c5bfe3a32c95755eaf8fa41918291 100644 (file)
@@ -236,9 +236,8 @@ class InconsistentObjChecker:
             log.info('shard = %r' % shard)
             log.info('attr = %s' % attr_name)
             assert 'osd' in shard
-            assert attr_name in shard
             osd = shard['osd']
-            attr = shard[attr_name]
+            attr = shard.get(attr_name, False)
             if osd == self.osd:
                 assert bad_attr is None, \
                     "multiple entries found for the given OSD"
@@ -302,10 +301,10 @@ def test_list_inconsistent_obj(ctx, manager, osd_remote, pg, acting, osd_id,
             with contextlib.closing(StringIO()) as out:
                 mon.run(args=cmd.split(), stdout=out)
                 objs = json.loads(out.getvalue())
-            assert len(objs) == 1
+            assert len(objs['inconsistents']) == 1
 
             checker = InconsistentObjChecker(osd_id, acting, obj_name)
-            inc_obj = objs[0]
+            inc_obj = objs['inconsistents'][0]
             log.info('inc = %r', inc_obj)
             checker.basic_checks(inc_obj)
             for check in checks: