[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2 for 2.0 2/2] qemu-iotests: Add TestRepairQuor
From: |
Benoît Canet |
Subject: |
Re: [Qemu-devel] [PATCH V2 for 2.0 2/2] qemu-iotests: Add TestRepairQuorum to 041 to test drive-mirror node-name mode. |
Date: |
Thu, 6 Mar 2014 17:09:37 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
The Thursday 06 Mar 2014 à 15:06:22 (+0100), Benoît Canet wrote :
> The the to-replace-node-name is designed to allow repairing of broken Quorum
> file.
> This patch introduce a new class TestRepairQuorum testing that the feature
> works.
> Some further work will be done on QEMU to improve the robutness of the tests.
>
> Signed-off-by: Benoit Canet <address@hidden>
> ---
> tests/qemu-iotests/041 | 221
> ++++++++++++++++++++++++++++++++++++++++++++-
> tests/qemu-iotests/041.out | 4 +-
> 2 files changed, 219 insertions(+), 6 deletions(-)
>
> diff --git a/tests/qemu-iotests/041 b/tests/qemu-iotests/041
> index ec470b2..9624e95 100755
> --- a/tests/qemu-iotests/041
> +++ b/tests/qemu-iotests/041
> @@ -28,6 +28,12 @@ target_backing_img = os.path.join(iotests.test_dir,
> 'target-backing.img')
> test_img = os.path.join(iotests.test_dir, 'test.img')
> target_img = os.path.join(iotests.test_dir, 'target.img')
>
> +quorum_img1 = os.path.join(iotests.test_dir, 'quorum1.img')
> +quorum_img2 = os.path.join(iotests.test_dir, 'quorum2.img')
> +quorum_img3 = os.path.join(iotests.test_dir, 'quorum3.img')
> +quorum_repair_img = os.path.join(iotests.test_dir, 'quorum_repair.img')
> +quorum_snapshot_file = os.path.join(iotests.test_dir, 'quorum_snapshot.img')
> +
> class ImageMirroringTestCase(iotests.QMPTestCase):
> '''Abstract base class for image mirroring test cases'''
>
> @@ -42,8 +48,8 @@ class ImageMirroringTestCase(iotests.QMPTestCase):
> ready = True
>
> def wait_ready_and_cancel(self, drive='drive0'):
> - self.wait_ready(drive)
> - event = self.cancel_and_wait()
> + self.wait_ready(drive=drive)
> + event = self.cancel_and_wait(drive=drive)
> self.assertEquals(event['event'], 'BLOCK_JOB_COMPLETED')
> self.assert_qmp(event, 'data/type', 'mirror')
> self.assert_qmp(event, 'data/offset', self.image_len)
> @@ -52,12 +58,12 @@ class ImageMirroringTestCase(iotests.QMPTestCase):
> def complete_and_wait(self, drive='drive0', wait_ready=True):
> '''Complete a block job and wait for it to finish'''
> if wait_ready:
> - self.wait_ready()
> + self.wait_ready(drive=drive)
>
> result = self.vm.qmp('block-job-complete', device=drive)
> self.assert_qmp(result, 'return', {})
>
> - event = self.wait_until_completed()
> + event = self.wait_until_completed(drive=drive)
> self.assert_qmp(event, 'data/type', 'mirror')
>
> class TestSingleDrive(ImageMirroringTestCase):
> @@ -718,5 +724,212 @@ class TestUnbackedSource(ImageMirroringTestCase):
> self.complete_and_wait()
> self.assert_no_active_block_jobs()
>
> +class TestRepairQuorum(ImageMirroringTestCase):
> + """ This class test quorum file repair using drive-mirror.
> + It's mostly a fork of TestSingleDrive """
> + image_len = 1 * 1024 * 1024 # MB
> + IMAGES = [ quorum_img1, quorum_img2, quorum_img3 ]
> +
> + def setUp(self):
> + self.vm = iotests.VM()
> +
> + # Add each individual quorum images
> + for i in self.IMAGES:
> + qemu_img('create', '-f', iotests.imgfmt, i,
> + str(TestSingleDrive.image_len))
> + # Assign a node name to each quorum image in order to manipulate
> + # them
> + opts = "node-name=img%i" % self.IMAGES.index(i)
> + self.vm = self.vm.add_drive(i, opts)
> +
> + self.vm.launch()
> +
> + #assemble the quorum block device from the individual files
> + args = { "options" : { "driver": "quorum", "id": "quorum0",
> + "vote-threshold": 2, "children": [ "img0", "img1", "img2" ]
> } }
> + result = self.vm.qmp("blockdev-add", **args)
> + self.assert_qmp(result, 'return', {})
> +
> +
> + def tearDown(self):
> + self.vm.shutdown()
> + for i in self.IMAGES + [ quorum_repair_img ]:
> + # Do a try/except because the test may have deleted some images
> + try:
> + os.remove(i)
> + except OSError:
> + pass
> +
> + def test_complete(self):
> + self.assert_no_active_block_jobs()
> +
> + result = self.vm.qmp('drive-mirror', device='quorum0', sync='full',
> + new_node_name="repair0",
> + to_replace_node_name="img1",
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'return', {})
> +
> + self.complete_and_wait(drive="quorum0")
> + result = self.vm.qmp('query-named-block-nodes')
> + self.assert_qmp(result, 'return[0]/file', quorum_repair_img)
> + # TODO: a better test requiring some QEMU infrastructure will be
> added
> + # to check that this file is really driven by quorum
> + self.vm.shutdown()
> + self.assertTrue(iotests.compare_images(quorum_img2,
> quorum_repair_img),
> + 'target image does not match source after mirroring')
> +
> + def test_cancel(self):
> + self.assert_no_active_block_jobs()
> +
> + result = self.vm.qmp('drive-mirror', device='quorum0', sync='full',
> + new_node_name="repair0",
> + to_replace_node_name="img1",
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'return', {})
> +
> + self.cancel_and_wait(drive="quorum0", force=True)
> + # here we check that the last registered quorum file has not been
> + # swapped out and unref
> + result = self.vm.qmp('query-named-block-nodes')
> + self.assert_qmp(result, 'return[0]/file', quorum_img3)
> + self.vm.shutdown()
> +
> + def test_cancel_after_ready(self):
> + self.assert_no_active_block_jobs()
> +
> + result = self.vm.qmp('drive-mirror', device='quorum0', sync='full',
> + new_node_name="repair0",
> + to_replace_node_name="img1",
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'return', {})
> +
> + self.wait_ready_and_cancel(drive="quorum0")
> + result = self.vm.qmp('query-named-block-nodes')
> + # here we check that the last registered quorum file has not been
> + # swapped out and unref
> + self.assert_qmp(result, 'return[0]/file', quorum_img3)
> + self.vm.shutdown()
> + self.assertTrue(iotests.compare_images(quorum_img2,
> quorum_repair_img),
> + 'target image does not match source after mirroring')
> +
> + def test_pause(self):
> + self.assert_no_active_block_jobs()
> +
> + result = self.vm.qmp('drive-mirror', device='quorum0', sync='full',
> + new_node_name="repair0",
> + to_replace_node_name="img1",
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'return', {})
> +
> + result = self.vm.qmp('block-job-pause', device='quorum0')
> + self.assert_qmp(result, 'return', {})
> +
> + time.sleep(1)
> + result = self.vm.qmp('query-block-jobs')
> + offset = self.dictpath(result, 'return[0]/offset')
> +
> + time.sleep(1)
> + result = self.vm.qmp('query-block-jobs')
> + self.assert_qmp(result, 'return[0]/offset', offset)
> +
> + result = self.vm.qmp('block-job-resume', device='quorum0')
> + self.assert_qmp(result, 'return', {})
> +
> + self.complete_and_wait(drive="quorum0")
> + self.vm.shutdown()
> + self.assertTrue(iotests.compare_images(quorum_img2,
> quorum_repair_img),
> + 'target image does not match source after mirroring')
> +
> + def test_medium_not_found(self):
> + result = self.vm.qmp('drive-mirror', device='ide1-cd0', sync='full',
> + new_node_name='repair0',
> + to_replace_node_name='img1',
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'error/class', 'GenericError')
> +
> + def test_image_not_found(self):
> + result = self.vm.qmp('drive-mirror', device='quorum0', sync='full',
> + new_node_name='repair0',
> + to_replace_node_name='img1',
> + mode='existing',
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'error/class', 'GenericError')
> +
> + def test_device_not_found(self):
> + result = self.vm.qmp('drive-mirror', device='nonexistent',
> sync='full',
> + new_node_name='repair0',
> + to_replace_node_name='img1',
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'error/class', 'DeviceNotFound')
> +
> + def test_wrong_sync_mode(self):
> + result = self.vm.qmp('drive-mirror', device='quorum0',
> + new_node_name='repair0',
> + to_replace_node_name='img1',
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'error/class', 'GenericError')
> +
> + def test_no_new_node_name(self):
> + result = self.vm.qmp('drive-mirror', device='quorum0', sync='full',
> + to_replace_node_name='img1',
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'error/class', 'GenericError')
> +
> + def test_unexistant_to_replace_node_name(self):
> + result = self.vm.qmp('drive-mirror', device='quorum0', sync='full',
> + new_node_name='repair0',
> + to_replace_node_name='img77',
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'error/class', 'GenericError')
> +
> + def test_after_a_quorum_snapshot(self):
> + result = self.vm.qmp('blockdev-snapshot-sync', node_name='img1',
> + snapshot_file=quorum_snapshot_file,
> + snapshot_node_name="snap1");
> +
> + result = self.vm.qmp('drive-mirror', device='quorum0', sync='full',
> + new_node_name='repair0',
> + to_replace_node_name="img1",
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'error/class', 'GenericError')
> +
> + result = self.vm.qmp('drive-mirror', device='quorum0', sync='full',
> + new_node_name='repair0',
> + to_replace_node_name="snap1",
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'return', {})
> +
> + self.complete_and_wait(drive="quorum0")
> + result = self.vm.qmp('query-named-block-nodes')
> + self.assert_qmp(result, 'return[0]/file', quorum_repair_img)
> + # TODO: a better test requiring some QEMU infrastructure will be
> added
> + # to check that this file is really driven by quorum
> + self.vm.shutdown()
> +
> + def test_to_replace_node_name_is_in_use_while_mirroring(self):
> + self.assert_no_active_block_jobs()
> +
> + result = self.vm.qmp('drive-mirror', device='quorum0', sync='full',
> + new_node_name="repair0",
> + to_replace_node_name="img1",
> + target=quorum_repair_img, format=iotests.imgfmt)
> + self.assert_qmp(result, 'return', {})
> +
> + result = self.vm.qmp('block-job-pause', device='quorum0')
> + self.assert_qmp(result, 'return', {})
> +
> + result = self.vm.qmp('blockdev-snapshot-sync', node_name='img1',
> + snapshot_file=quorum_snapshot_file,
> + snapshot_node_name="snap1");
> + self.assert_qmp(result, 'error/class', 'GenericError')
> +
> + result = self.vm.qmp('block-job-resume', device='quorum0')
> + self.assert_qmp(result, 'return', {})
> +
> + self.complete_and_wait(drive="quorum0")
> + self.vm.shutdown()
> + self.assertTrue(iotests.compare_images(quorum_img2,
> quorum_repair_img),
> + 'target image does not match source after mirroring')
> +
> if __name__ == '__main__':
> iotests.main(supported_fmts=['qcow2', 'qed'])
> diff --git a/tests/qemu-iotests/041.out b/tests/qemu-iotests/041.out
> index 6d9bee1..12e9ab7 100644
> --- a/tests/qemu-iotests/041.out
> +++ b/tests/qemu-iotests/041.out
> @@ -1,5 +1,5 @@
> -...........................
> +.......................................
> ----------------------------------------------------------------------
> -Ran 27 tests
> +Ran 39 tests
>
> OK
> --
> 1.8.3.2
>
>
I realized that quorum is not builtin by default and this test cases should go
in their own file. Will do this.
Best regards
Benoît