diff options
author | Max Reitz <mreitz@redhat.com> | 2019-07-19 11:26:17 +0200 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2019-07-19 13:19:17 +0200 |
commit | 4687133b812323b743e490a21510a1e1ac0fb6df (patch) | |
tree | 0b60f087799cf87fcbbc5c7181984bd9772870f1 | |
parent | 61ad631cee21f878540681274fe0f53e7ee9f59e (diff) | |
download | qemu-4687133b812323b743e490a21510a1e1ac0fb6df.zip qemu-4687133b812323b743e490a21510a1e1ac0fb6df.tar.gz qemu-4687133b812323b743e490a21510a1e1ac0fb6df.tar.bz2 |
iotests: Add @has_quit to vm.shutdown()
If a test has issued a quit command already (which may be useful to do
explicitly because the test wants to show its effects),
QEMUMachine.shutdown() should not do so again. Otherwise, the VM may
well return an ECONNRESET which will lead QEMUMachine.shutdown() to
killing it, which then turns into a "qemu received signal 9" line.
Signed-off-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
-rw-r--r-- | python/qemu/machine.py | 5 | ||||
-rwxr-xr-x | tests/qemu-iotests/255 | 2 |
2 files changed, 4 insertions, 3 deletions
diff --git a/python/qemu/machine.py b/python/qemu/machine.py index 49445e6..128a3d1 100644 --- a/python/qemu/machine.py +++ b/python/qemu/machine.py @@ -329,13 +329,14 @@ class QEMUMachine(object): self._load_io_log() self._post_shutdown() - def shutdown(self): + def shutdown(self, has_quit=False): """ Terminate the VM and clean up """ if self.is_running(): try: - self._qmp.cmd('quit') + if not has_quit: + self._qmp.cmd('quit') self._qmp.close() except: self._popen.kill() diff --git a/tests/qemu-iotests/255 b/tests/qemu-iotests/255 index 49433ec..3632d50 100755 --- a/tests/qemu-iotests/255 +++ b/tests/qemu-iotests/255 @@ -132,4 +132,4 @@ with iotests.FilePath('src.qcow2') as src_path, \ vm.qmp_log('block-job-cancel', device='job0') vm.qmp_log('quit') - vm.shutdown() + vm.shutdown(has_quit=True) |