[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v5 32/45] iotests: add blockdev-add-transaction
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
[PATCH v5 32/45] iotests: add blockdev-add-transaction |
Date: |
Thu, 31 Mar 2022 00:28:49 +0300 |
Add a test for transaction support of blockdev-add.
Test is format-agnostic, so limit it to qcow2 to avoid extra test runs.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@openvz.org>
---
.../tests/blockdev-add-transaction | 52 +++++++++++++++++++
.../tests/blockdev-add-transaction.out | 6 +++
2 files changed, 58 insertions(+)
create mode 100755 tests/qemu-iotests/tests/blockdev-add-transaction
create mode 100644 tests/qemu-iotests/tests/blockdev-add-transaction.out
diff --git a/tests/qemu-iotests/tests/blockdev-add-transaction
b/tests/qemu-iotests/tests/blockdev-add-transaction
new file mode 100755
index 0000000000..ce3c1c069b
--- /dev/null
+++ b/tests/qemu-iotests/tests/blockdev-add-transaction
@@ -0,0 +1,52 @@
+#!/usr/bin/env python3
+#
+# Test blockdev-add transaction action
+#
+# Copyright (c) 2022 Virtuozzo International GmbH.
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+#
+
+import iotests
+from iotests import log
+
+iotests.script_initialize(supported_fmts=['qcow2'])
+
+with iotests.VM() as vm:
+ vm.launch()
+
+ # Use same node-name for nodes, neither one should appear.
+ vm.qmp_log('transaction', actions=[
+ {'type': 'blockdev-add',
+ 'data': {'node-name': 'node0', 'driver': 'null-co',
+ 'size': 1024 * 1024}},
+ {'type': 'blockdev-add',
+ 'data': {'node-name': 'node0', 'driver': 'null-co',
+ 'size': 1024 * 1024}}
+ ])
+
+ n = len(vm.qmp('query-named-block-nodes')['return'])
+ log(f'Created {n} nodes')
+
+ vm.qmp_log('transaction', actions=[
+ {'type': 'blockdev-add',
+ 'data': {'node-name': 'node0', 'driver': 'null-co',
+ 'size': 1024 * 1024}},
+ {'type': 'blockdev-add',
+ 'data': {'node-name': 'node1', 'driver': 'null-co',
+ 'size': 1024 * 1024}}
+ ])
+
+ n = len(vm.qmp('query-named-block-nodes')['return'])
+ log(f'Created {n} nodes')
diff --git a/tests/qemu-iotests/tests/blockdev-add-transaction.out
b/tests/qemu-iotests/tests/blockdev-add-transaction.out
new file mode 100644
index 0000000000..7e6cd5a9a3
--- /dev/null
+++ b/tests/qemu-iotests/tests/blockdev-add-transaction.out
@@ -0,0 +1,6 @@
+{"execute": "transaction", "arguments": {"actions": [{"data": {"driver":
"null-co", "node-name": "node0", "size": 1048576}, "type": "blockdev-add"},
{"data": {"driver": "null-co", "node-name": "node0", "size": 1048576}, "type":
"blockdev-add"}]}}
+{"error": {"class": "GenericError", "desc": "Duplicate nodes with
node-name='node0'"}}
+Created 0 nodes
+{"execute": "transaction", "arguments": {"actions": [{"data": {"driver":
"null-co", "node-name": "node0", "size": 1048576}, "type": "blockdev-add"},
{"data": {"driver": "null-co", "node-name": "node1", "size": 1048576}, "type":
"blockdev-add"}]}}
+{"return": {}}
+Created 2 nodes
--
2.35.1
- [PATCH v5 25/45] blockdev: qmp_transaction: refactor loop to classic for, (continued)
- [PATCH v5 25/45] blockdev: qmp_transaction: refactor loop to classic for, Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 30/45] block: bdrv_insert_node(): use BDRV_O_NOPERM, Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 33/45] block-backend: blk_root(): drop const specifier on return type, Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 29/45] block: introduce BDRV_O_NOPERM flag, Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 36/45] block: bdrv_replace_child_bs(): move to external transaction, Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 39/45] block: bdrv_get_xdbg_block_graph(): report export ids, Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 42/45] iotests.py: add VM.qmp_check() helper, Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 43/45] iotests: add filter-insertion, Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 27/45] blockdev: qmp_transaction: drop extra generic layer, Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 44/45] block: bdrv_open_inherit: create BlockBackend only when necessary, Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 32/45] iotests: add blockdev-add-transaction,
Vladimir Sementsov-Ogievskiy <=
- [PATCH v5 16/45] block: drop bdrv_detach_child(), Vladimir Sementsov-Ogievskiy, 2022/03/30
- [PATCH v5 34/45] block/export: add blk_by_export_id(), Vladimir Sementsov-Ogievskiy, 2022/03/30