[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 21/22] iotests: Check that @replaces can replace filters
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [PATCH 21/22] iotests: Check that @replaces can replace filters |
Date: |
Thu, 26 Sep 2019 14:42:06 +0000 |
20.09.2019 18:28, Max Reitz wrote:
> Signed-off-by: Max Reitz <address@hidden>
> ---
> tests/qemu-iotests/041 | 45 ++++++++++++++++++++++++++++++++++++++
> tests/qemu-iotests/041.out | 4 ++--
> 2 files changed, 47 insertions(+), 2 deletions(-)
>
> diff --git a/tests/qemu-iotests/041 b/tests/qemu-iotests/041
> index 148dc47ce4..e4cc829fe2 100755
> --- a/tests/qemu-iotests/041
> +++ b/tests/qemu-iotests/041
> @@ -1220,6 +1220,51 @@ class TestOrphanedSource(iotests.QMPTestCase):
> self.assertFalse('mirror-filter' in nodes,
> 'Mirror filter node did not disappear')
>
> +# Test cases for @replaces that do not necessarily involve Quorum
> +class TestReplaces(iotests.QMPTestCase):
> + # Each of these test cases needs their own block graph, so do not
> + # create any nodes here
> + def setUp(self):
> + self.vm = iotests.VM()
> + self.vm.launch()
> +
> + def tearDown(self):
> + self.vm.shutdown()
> + for img in (test_img, target_img):
> + try:
> + os.remove(img)
> + except OSError:
> + pass
dead code, but may be used in future patch..
> +
> + '''
> + Check that we can replace filter nodes.
> + '''
> + def test_replace_filter(self):
> + result = self.vm.qmp('blockdev-add', **{
> + 'driver': 'copy-on-read',
> + 'node-name': 'filter0',
> + 'file': {
> + 'driver': 'copy-on-read',
> + 'node-name': 'filter1',
> + 'file': {
> + 'driver': 'null-co'
> + }
> + }
> + })
> + self.assert_qmp(result, 'return', {})
> +
> + result = self.vm.qmp('blockdev-add',
> + node_name='target', driver='null-co')
> + self.assert_qmp(result, 'return', {})
> +
> + result = self.vm.qmp('blockdev-mirror', job_id='mirror',
> device='filter0',
> + target='target', sync='full',
> replaces='filter1')
> + self.assert_qmp(result, 'return', {})
> +
> + self.complete_and_wait('mirror')
> +
> + self.vm.assert_block_path('filter0/file', 'target')
> +
> if __name__ == '__main__':
> iotests.main(supported_fmts=['qcow2', 'qed'],
> supported_protocols=['file'])
> diff --git a/tests/qemu-iotests/041.out b/tests/qemu-iotests/041.out
> index ffc779b4d1..877b76fd31 100644
> --- a/tests/qemu-iotests/041.out
> +++ b/tests/qemu-iotests/041.out
> @@ -1,5 +1,5 @@
> -.............................................................................................
> +..............................................................................................
> ----------------------------------------------------------------------
> -Ran 93 tests
> +Ran 94 tests
>
> OK
>
Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
--
Best regards,
Vladimir
- Re: [PATCH 14/22] quorum: Stop marking it as a filter, (continued)
- [PATCH 15/22] mirror: Prevent loops, Max Reitz, 2019/09/20
- [PATCH 16/22] iotests: Use complete_and_wait() in 155, Max Reitz, 2019/09/20
- [PATCH 18/22] iotests: Resolve TODOs in 041, Max Reitz, 2019/09/20
- [PATCH 19/22] iotests: Use self.image_len in TestRepairQuorum, Max Reitz, 2019/09/20
- [PATCH 21/22] iotests: Check that @replaces can replace filters, Max Reitz, 2019/09/20
- Re: [PATCH 21/22] iotests: Check that @replaces can replace filters,
Vladimir Sementsov-Ogievskiy <=
- [PATCH 22/22] iotests: Mirror must not attempt to create loops, Max Reitz, 2019/09/20
- [PATCH 20/22] iotests: Add tests for invalid Quorum @replaces, Max Reitz, 2019/09/20