qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 17/25] iotests/mirror-top-perms: switch to AQMP


From: Vladimir Sementsov-Ogievskiy
Subject: Re: [PATCH v2 17/25] iotests/mirror-top-perms: switch to AQMP
Date: Thu, 16 Dec 2021 14:01:28 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0

15.12.2021 22:39, John Snow wrote:
Signed-off-by: John Snow <jsnow@redhat.com>


Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>


---

Note: I still need to adjust the logging. The problem now is that the
logging messages include the PID of the test process, so they need to be
filtered out. I'll investigate that for a follow-up, or for v2.

I could just add yet another filtering function somewhere, but I think
it's getting out of hand with how many filters and loggers there are, so
I want to give it a slightly more serious treatment instead of a
hackjob.

Signed-off-by: John Snow <jsnow@redhat.com>

Hmm, something is wrong with your scripts or git-notes practices )

---
  tests/qemu-iotests/tests/mirror-top-perms | 7 +++----
  1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/tests/qemu-iotests/tests/mirror-top-perms 
b/tests/qemu-iotests/tests/mirror-top-perms
index 0a51a613f3..f394931a00 100755
--- a/tests/qemu-iotests/tests/mirror-top-perms
+++ b/tests/qemu-iotests/tests/mirror-top-perms
@@ -23,7 +23,6 @@ import os
from qemu.aqmp import ConnectError
  from qemu.machine import machine
-from qemu.qmp import QMPConnectError
import iotests
  from iotests import change_log_level, qemu_img
@@ -101,13 +100,13 @@ class TestMirrorTopPerms(iotests.QMPTestCase):
          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.
+            # TODO: Remove change_log_level and allow the errors to be logged.
+            #       This necessitates a PID filter on *all* logging output.
              with change_log_level('qemu.aqmp'):
                  self.vm_b.launch()
                  print('ERROR: VM B launched successfully, '
                        'this should not have happened')
-        except (QMPConnectError, ConnectError):
+        except ConnectError:
              assert 'Is another process using the image' in self.vm_b.get_log()
result = self.vm.qmp('block-job-cancel',



--
Best regards,
Vladimir



reply via email to

[Prev in Thread] Current Thread [Next in Thread]