19.06.2020 22:56, Eric Blake wrote:
From: John Snow <js...@redhat.com>

Signed-off-by: John Snow <js...@redhat.com>
Message-Id: <20200514034922.24834-5-js...@redhat.com>
Signed-off-by: Eric Blake <ebl...@redhat.com>
---
  tests/qemu-iotests/257        | 110 +---------------------------
  tests/qemu-iotests/bitmaps.py | 131 ++++++++++++++++++++++++++++++++++
  2 files changed, 132 insertions(+), 109 deletions(-)
  create mode 100644 tests/qemu-iotests/bitmaps.py

diff --git a/tests/qemu-iotests/257 b/tests/qemu-iotests/257
index 004a433b8be2..2a81f9e30c56 100755
--- a/tests/qemu-iotests/257
+++ b/tests/qemu-iotests/257
@@ -24,120 +24,12 @@ import os

  import iotests
  from iotests import log, qemu_img
+from bitmaps import EmulatedBitmap, GROUPS

Exporting global variable of some unknown to this module structure doesn't seem
to be good module design. One day we may want to refactor this.. For now:

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


--
Best regards,
Vladimir

Reply via email to