laforge submitted this change.

View Change


Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve
vty_test_runner: Don't wait for more than 10s when connecting to sockets

Change-Id: Iadb437ce0bafb28baa978d287dbb1006ec97f236
---
M tests/vty/vty_test_runner.py
1 file changed, 13 insertions(+), 0 deletions(-)

diff --git a/tests/vty/vty_test_runner.py b/tests/vty/vty_test_runner.py
index fce3654..ba1d90b 100755
--- a/tests/vty/vty_test_runner.py
+++ b/tests/vty/vty_test_runner.py
@@ -30,6 +30,8 @@
# to be able to find $top_srcdir/doc/...
confpath = os.path.join(sys.path[0], '..')

+TIMEOUT = 10
+
class TestVTYBase(unittest.TestCase):

def checkForEndAndExit(self):
@@ -127,6 +129,7 @@
# IPv4:
s = socket.socket(socket.AF_INET, socket.SOCK_STREAM, proto)
s.bind(('127.0.0.3', 0))
+ s.settimeout(TIMEOUT)
try:
s.connect(('127.0.0.2',2905))
except socket.error as msg:
@@ -137,6 +140,7 @@
# IPv6:
s = socket.socket(socket.AF_INET6, socket.SOCK_STREAM, proto)
s.bind(('::1', 0))
+ s.settimeout(TIMEOUT)
try:
s.connect(('::1',2905))
except socket.error as msg:

To view, visit change 35940. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: Iadb437ce0bafb28baa978d287dbb1006ec97f236
Gerrit-Change-Number: 35940
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged