kgiusti commented on a change in pull request #1516:
URL: https://github.com/apache/qpid-dispatch/pull/1516#discussion_r807140454
##########
File path: tests/system_tests_http1_adaptor.py
##########
@@ -905,7 +918,7 @@ def test_02_backpressure_server(self):
client_sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
Review comment:
Yeah while we don't want to leak sockets this particular set of tests
re-create the router and all the ephemeral ports for each test. That should
avoid the "socket in use" errors we've seen on other tests since in those
tests the TestServers use the same port # for all tests (since the router HTTP
server-facing connection configuration doesn't change for each individual
sub-test). When one of those TestServers is left running from a previous test
things get ugly.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
