Change in vdsm[master]: ssl: ssl_accept blocks after reboot

danken at redhat.com danken at redhat.com
Wed Oct 1 12:45:23 UTC 2014


Dan Kenigsberg has posted comments on this change.

Change subject: ssl: ssl_accept blocks after reboot
......................................................................


Patch Set 2: Code-Review-1

(2 comments)

http://gerrit.ovirt.org/#/c/33611/2/tests/sslTests.py
File tests/sslTests.py:

Line 109: 
Line 110: class SocketTests(TestCaseBase):
Line 111: 
Line 112:     def test_block_socket(self):
Line 113:         # this test make sure that we won't block during
I don't understand how this test works :-(

You start a server, open a TCP connection to it, but you don't check that the server is still responsive.

Would this test really fail on master branch (before your fix)?

I think that you should:
* start a server
* open a DoS connection to it
* open a legitimate client to the server
* if the client is not served within 10 seconds, fail the test.
Line 114:         # accept. It will hang the build if accept blocks
Line 115:         server = TestServer()
Line 116:         server.start()
Line 117:         try:


Line 119:             sock.settimeout(1)
Line 120:             sock.connect((HOST, server.port))
Line 121:         finally:
Line 122:             server.stop()
Line 123:             sock.close()
puritan: at this point, "sock" may have not been defined. better use

  with closing(socket.socket(socket.AF_INET, socket.SOCK_STREAM)) as sock:
Line 124: 
Line 125: 
Line 126: class VerifyingTransportTests(TestCaseBase):
Line 127: 


-- 
To view, visit http://gerrit.ovirt.org/33611
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I759436b5bfb6c2334d253d12806258cbe1c3720f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list