[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 03/17] iotests/mirror-top-perms: switch to AQMP
From: |
John Snow |
Subject: |
[PULL 03/17] iotests/mirror-top-perms: switch to AQMP |
Date: |
Thu, 21 Apr 2022 11:15:46 -0400 |
We don't have to maintain compatibility with both QMP libraries anymore,
so we can just remove the old exception. While we're here, take
advantage of the extra fields present in the VMLaunchFailure exception
that machine.py now raises.
(Note: I'm leaving the logging suppression here unchanged. I had
suggested previously we use filters to scrub the PID out of the logging
information so it could just be diffed as part of the iotest output, but
that meant *always* scrubbing PID from logger output, which defeated the
point of even offering that information in the output to begin with.
Ultimately, I decided it's fine to just suppress the logger temporarily.)
Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Acked-by: Hanna Reitz <hreitz@redhat.com>
Message-id: 20220321203315.909411-4-jsnow@redhat.com
Signed-off-by: John Snow <jsnow@redhat.com>
---
tests/qemu-iotests/tests/mirror-top-perms | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/tests/qemu-iotests/tests/mirror-top-perms
b/tests/qemu-iotests/tests/mirror-top-perms
index 6ac8d5efccb..a9f275cd7f2 100755
--- a/tests/qemu-iotests/tests/mirror-top-perms
+++ b/tests/qemu-iotests/tests/mirror-top-perms
@@ -22,7 +22,6 @@
import os
from qemu.machine import machine
-from qemu.qmp import QMPConnectError
import iotests
from iotests import change_log_level, qemu_img
@@ -98,15 +97,13 @@ class TestMirrorTopPerms(iotests.QMPTestCase):
self.vm_b.add_blockdev(f'file,node-name=drive0,filename={source}')
self.vm_b.add_device('virtio-blk,drive=drive0,share-rw=on')
try:
- # Silence AQMP errors temporarily.
- # TODO: Remove this and just allow the errors to be logged when
- # AQMP fully replaces QMP.
+ # Silence AQMP logging errors temporarily.
with change_log_level('qemu.aqmp'):
self.vm_b.launch()
print('ERROR: VM B launched successfully, '
'this should not have happened')
- except (QMPConnectError, machine.VMLaunchFailure):
- assert 'Is another process using the image' in self.vm_b.get_log()
+ except machine.VMLaunchFailure as exc:
+ assert 'Is another process using the image' in exc.output
result = self.vm.qmp('block-job-cancel',
device='mirror')
--
2.34.1
- [PULL 00/17] Python patches, John Snow, 2022/04/21
- [PULL 02/17] scripts/bench-block-job: switch to AQMP, John Snow, 2022/04/21
- [PULL 07/17] python/qmp-shell: relicense as LGPLv2+, John Snow, 2022/04/21
- [PULL 05/17] python/aqmp: add explicit GPLv2 license to legacy.py, John Snow, 2022/04/21
- [PULL 03/17] iotests/mirror-top-perms: switch to AQMP,
John Snow <=
- [PULL 08/17] python/aqmp-tui: relicense as LGPLv2+, John Snow, 2022/04/21
- [PULL 06/17] python/aqmp: relicense as LGPLv2+, John Snow, 2022/04/21
- [PULL 12/17] python/aqmp: copy qmp docstrings to qemu.aqmp.legacy, John Snow, 2022/04/21
- [PULL 10/17] python/aqmp: take QMPBadPortError and parse_address from qemu.qmp, John Snow, 2022/04/21
- [PULL 11/17] python/aqmp: fully separate from qmp.QEMUMonitorProtocol, John Snow, 2022/04/21
- [PULL 14/17] python: re-enable pylint duplicate-code warnings, John Snow, 2022/04/21
- [PULL 17/17] python/qmp: remove pylint workaround from legacy.py, John Snow, 2022/04/21
- [PULL 13/17] python: remove the old QMP package, John Snow, 2022/04/21
- [PULL 01/17] python/machine: permanently switch to AQMP, John Snow, 2022/04/21
- [PULL 04/17] iotests: switch to AQMP, John Snow, 2022/04/21