qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH v2 00/11] iotests: use vm.cmd()


From: Vladimir Sementsov-Ogievskiy
Subject: Re: [PATCH v2 00/11] iotests: use vm.cmd()
Date: Mon, 19 Sep 2022 20:16:37 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.1

ping

On 6/6/22 10:27, Vladimir Sementsov-Ogievskiy wrote:
Hi all!

Let's get rid of pattern

     result = self.vm.qmp(...)
     self.assert_qmp(result, 'return', {})

And switch to just

     self.vm.cmd(...)

Supersedes: <20220408170214.45585-1-vsementsov@openvz.org>
    ([RFC 0/2] introduce QEMUMachind.cmd())

Vladimir Sementsov-Ogievskiy (11):
   python: rename QEMUMonitorProtocol.cmd() to cmd_raw()
   python/qemu: rename command() to cmd()
   python/machine.py: upgrade vm.cmd() method
   iotests: QemuStorageDaemon: add cmd() method like in QEMUMachine.
   iotests: add some missed checks of qmp result
   iotests: refactor some common qmp result checks into generic pattern
   iotests: drop some occasional semicolons
   iotests: drop some extra ** in qmp() call
   iotests.py: pause_job(): drop return value
   tests/vm/basevm.py: use cmd() instead of qmp()
   python: use vm.cmd() instead of vm.qmp() where appropriate

  docs/devel/testing.rst                        |  10 +-
  python/qemu/machine/machine.py                |  20 +-
  python/qemu/qmp/legacy.py                     |  10 +-
  python/qemu/qmp/qmp_shell.py                  |  13 +-
  python/qemu/utils/qemu_ga_client.py           |   2 +-
  python/qemu/utils/qom.py                      |   8 +-
  python/qemu/utils/qom_common.py               |   2 +-
  python/qemu/utils/qom_fuse.py                 |   6 +-
  scripts/cpu-x86-uarch-abi.py                  |   8 +-
  scripts/device-crash-test                     |   8 +-
  scripts/render_block_graph.py                 |   8 +-
  tests/avocado/avocado_qemu/__init__.py        |   4 +-
  tests/avocado/cpu_queries.py                  |   4 +-
  tests/avocado/hotplug_cpu.py                  |  10 +-
  tests/avocado/info_usernet.py                 |   4 +-
  tests/avocado/machine_arm_integratorcp.py     |   6 +-
  tests/avocado/machine_m68k_nextcube.py        |   4 +-
  tests/avocado/machine_mips_malta.py           |   6 +-
  tests/avocado/machine_s390_ccw_virtio.py      |  28 +-
  tests/avocado/migration.py                    |  10 +-
  tests/avocado/pc_cpu_hotplug_props.py         |   2 +-
  tests/avocado/version.py                      |   4 +-
  tests/avocado/virtio_check_params.py          |   6 +-
  tests/avocado/virtio_version.py               |   4 +-
  tests/avocado/vnc.py                          |  16 +-
  tests/avocado/x86_cpu_model_versions.py       |  10 +-
  tests/migration/guestperf/engine.py           | 150 +++---
  tests/qemu-iotests/030                        | 168 +++---
  tests/qemu-iotests/040                        | 171 +++----
  tests/qemu-iotests/041                        | 482 ++++++++----------
  tests/qemu-iotests/045                        |  15 +-
  tests/qemu-iotests/055                        |  62 +--
  tests/qemu-iotests/056                        |  77 ++-
  tests/qemu-iotests/093                        |  42 +-
  tests/qemu-iotests/118                        | 225 ++++----
  tests/qemu-iotests/124                        | 102 ++--
  tests/qemu-iotests/129                        |  14 +-
  tests/qemu-iotests/132                        |   5 +-
  tests/qemu-iotests/139                        |  45 +-
  tests/qemu-iotests/147                        |  30 +-
  tests/qemu-iotests/151                        |  56 +-
  tests/qemu-iotests/152                        |   8 +-
  tests/qemu-iotests/155                        |  55 +-
  tests/qemu-iotests/165                        |   8 +-
  tests/qemu-iotests/196                        |   3 +-
  tests/qemu-iotests/205                        |   6 +-
  tests/qemu-iotests/218                        | 105 ++--
  tests/qemu-iotests/245                        | 245 ++++-----
  tests/qemu-iotests/256                        |  34 +-
  tests/qemu-iotests/257                        |  36 +-
  tests/qemu-iotests/264                        |  31 +-
  tests/qemu-iotests/281                        |  21 +-
  tests/qemu-iotests/295                        |  16 +-
  tests/qemu-iotests/296                        |  21 +-
  tests/qemu-iotests/298                        |  13 +-
  tests/qemu-iotests/300                        |  54 +-
  tests/qemu-iotests/iotests.py                 |  18 +-
  .../tests/export-incoming-iothread            |   6 +-
  .../qemu-iotests/tests/graph-changes-while-io |   6 +-
  tests/qemu-iotests/tests/image-fleecing       |   3 +-
  .../tests/migrate-bitmaps-postcopy-test       |  31 +-
  tests/qemu-iotests/tests/migrate-bitmaps-test |  45 +-
  .../qemu-iotests/tests/migrate-during-backup  |  41 +-
  .../qemu-iotests/tests/migration-permissions  |   9 +-
  .../tests/mirror-ready-cancel-error           |  74 ++-
  tests/qemu-iotests/tests/mirror-top-perms     |  16 +-
  tests/qemu-iotests/tests/nbd-multiconn        |  12 +-
  tests/qemu-iotests/tests/reopen-file          |   3 +-
  .../qemu-iotests/tests/stream-error-on-reset  |   6 +-
  tests/vm/basevm.py                            |   4 +-
  70 files changed, 1174 insertions(+), 1613 deletions(-)



--
Best regards,
Vladimir



reply via email to

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