]> xenbits.xensource.com Git - people/pauldu/qemu.git/commitdiff
iotests: Let 045 be run concurrently
authorMax Reitz <mreitz@redhat.com>
Sun, 10 Feb 2019 14:57:35 +0000 (15:57 +0100)
committerMax Reitz <mreitz@redhat.com>
Mon, 25 Feb 2019 14:11:28 +0000 (15:11 +0100)
Adding a telnet monitor for no real purpose on a fixed port is not so
great.  Just use a null monitor instead.

Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: John Snow <jsnow@redhat.com>
Message-id: 20190210145736.1486-10-mreitz@redhat.com
Signed-off-by: Max Reitz <mreitz@redhat.com>
scripts/qemu.py
tests/qemu-iotests/045

index 32b00af5cca7eb564fa44341fcd758f1a0a0cfd5..f7269eefbb118d59e5399c540d721ed92121a363 100644 (file)
@@ -144,10 +144,9 @@ class QEMUMachine(object):
         return False
 
     # This can be used to add an unused monitor instance.
-    def add_monitor_telnet(self, ip, port):
-        args = 'tcp:%s:%d,server,nowait,telnet' % (ip, port)
+    def add_monitor_null(self):
         self._args.append('-monitor')
-        self._args.append(args)
+        self._args.append('null')
 
     def add_fd(self, fd, fdset, opaque, opts=''):
         """
index 55a5d31ca82856373404eb1505c377593d866371..d5484a0ee1779b3341501880981888c84193aa4a 100755 (executable)
@@ -132,7 +132,7 @@ class TestSCMFd(iotests.QMPTestCase):
         qemu_img('create', '-f', iotests.imgfmt, image0, '128K')
         # Add an unused monitor, to verify it works fine when two monitor
         # instances present
-        self.vm.add_monitor_telnet("0",4445)
+        self.vm.add_monitor_null()
         self.vm.launch()
 
     def tearDown(self):