Skip to content

Commit

Permalink
Acceptance tests: work around socket dir
Browse files Browse the repository at this point in the history
Change 32558ce introduced specific directories for the socket dir
when using python/qemu/machine.py:QEMUMachine.  iotests probably
didn't catch the condition that two simultaneous QEMUMachine
instances, without manually set temporary or socket dirs would clash.

Having two QEMUMachine instances is a condition expected for many
acceptance tests, and it's already used by the migration tests.

Signed-off-by: Cleber Rosa <crosa@redhat.com>
  • Loading branch information
clebergnu committed Oct 28, 2019
1 parent 1b6ee29 commit 5449d93
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tests/acceptance/avocado_qemu/__init__.py
Expand Up @@ -11,6 +11,7 @@
import os
import sys
import uuid
import tempfile

import avocado

Expand Down Expand Up @@ -69,7 +70,7 @@ def setUp(self):
self.cancel("No QEMU binary defined or found in the source tree")

def _new_vm(self, *args):
vm = QEMUMachine(self.qemu_bin)
vm = QEMUMachine(self.qemu_bin, sock_dir=tempfile.mkdtemp())
if args:
vm.add_args(*args)
return vm
Expand Down

0 comments on commit 5449d93

Please sign in to comment.