|
 |
9bac43 |
From 2f7ffaddd0a233a233125320b6595a091c19c18f Mon Sep 17 00:00:00 2001
|
|
|
9bac43 |
From: Eric Blake <eblake@redhat.com>
|
|
|
9bac43 |
Date: Fri, 6 Oct 2017 19:24:07 +0200
|
|
|
9bac43 |
Subject: [PATCH 15/34] qemu-iotests: improve nbd-fault-injector.py startup
|
|
|
9bac43 |
protocol
|
|
|
9bac43 |
|
|
|
9bac43 |
RH-Author: Eric Blake <eblake@redhat.com>
|
|
|
9bac43 |
Message-id: <20171006192409.29915-3-eblake@redhat.com>
|
|
|
9bac43 |
Patchwork-id: 76914
|
|
|
9bac43 |
O-Subject: [RHEV-7.5 qemu-kvm-rhev PATCH 2/4] qemu-iotests: improve nbd-fault-injector.py startup protocol
|
|
|
9bac43 |
Bugzilla: 1482478
|
|
|
9bac43 |
RH-Acked-by: Max Reitz <mreitz@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
9bac43 |
|
|
|
9bac43 |
From: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
9bac43 |
|
|
|
9bac43 |
Currently 083 waits for the nbd-fault-injector.py server to start up by
|
|
|
9bac43 |
looping until netstat shows the TCP listen socket.
|
|
|
9bac43 |
|
|
|
9bac43 |
The startup protocol can be simplified by passing a 0 port number to
|
|
|
9bac43 |
nbd-fault-injector.py. The kernel will allocate a port in bind(2) and
|
|
|
9bac43 |
the final port number can be printed by nbd-fault-injector.py.
|
|
|
9bac43 |
|
|
|
9bac43 |
This should make it slightly nicer and less TCP-specific to wait for
|
|
|
9bac43 |
server startup. This patch changes nbd-fault-injector.py, the next one
|
|
|
9bac43 |
will rewrite server startup in 083.
|
|
|
9bac43 |
|
|
|
9bac43 |
Reviewed-by: Eric Blake <eblake@redhat.com>
|
|
|
9bac43 |
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
9bac43 |
Message-Id: <20170829122745.14309-3-stefanha@redhat.com>
|
|
|
9bac43 |
Signed-off-by: Eric Blake <eblake@redhat.com>
|
|
|
9bac43 |
(cherry picked from commit 6e592fc92234a58c7156c385840633c17dedd24f)
|
|
|
9bac43 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9bac43 |
---
|
|
|
9bac43 |
tests/qemu-iotests/nbd-fault-injector.py | 4 ++++
|
|
|
9bac43 |
1 file changed, 4 insertions(+)
|
|
|
9bac43 |
|
|
|
9bac43 |
diff --git a/tests/qemu-iotests/nbd-fault-injector.py b/tests/qemu-iotests/nbd-fault-injector.py
|
|
|
9bac43 |
index 6c07191..1c10dcb 100755
|
|
|
9bac43 |
--- a/tests/qemu-iotests/nbd-fault-injector.py
|
|
|
9bac43 |
+++ b/tests/qemu-iotests/nbd-fault-injector.py
|
|
|
9bac43 |
@@ -235,11 +235,15 @@ def open_socket(path):
|
|
|
9bac43 |
sock = socket.socket()
|
|
|
9bac43 |
sock.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1)
|
|
|
9bac43 |
sock.bind((host, int(port)))
|
|
|
9bac43 |
+
|
|
|
9bac43 |
+ # If given port was 0 the final port number is now available
|
|
|
9bac43 |
+ path = '%s:%d' % sock.getsockname()
|
|
|
9bac43 |
else:
|
|
|
9bac43 |
sock = socket.socket(socket.AF_UNIX)
|
|
|
9bac43 |
sock.bind(path)
|
|
|
9bac43 |
sock.listen(0)
|
|
|
9bac43 |
print 'Listening on %s' % path
|
|
|
9bac43 |
+ sys.stdout.flush() # another process may be waiting, show message now
|
|
|
9bac43 |
return sock
|
|
|
9bac43 |
|
|
|
9bac43 |
def usage(args):
|
|
|
9bac43 |
--
|
|
|
9bac43 |
1.8.3.1
|
|
|
9bac43 |
|