Zhou Zheng Sheng has posted comments on this change.

Change subject: avoid creating hangover in storage domain cache after formatting
......................................................................


Patch Set 7: Verified; Looks good to me, but someone else must approve

Rebase and resolve conflicts 
Test the patch myself
Copy Vinzenz Feenstra's +1 from patch set 6

Jenkins Build fails is not related to this patch, see 
http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/30/console and you 
will see.

ERROR: testConnectWithoutCertificateFails (sslTests.SSLTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File 
"/home/jenkins/workspace/vdsm_unit_tests_manual_gerrit/tests/sslTests.py", line 
107, in setUp
    self.server.bind(ADDRESS)
  File "/usr/lib64/python2.7/site-packages/M2Crypto/SSL/Connection.py", line 
89, in bind
    self.socket.bind(addr)
  File "/usr/lib64/python2.7/socket.py", line 224, in meth
    return getattr(self._sock,name)(*args)
error: [Errno 98] Address already in use

It looks like there are parallel builds in progress to race the binding address 
of SSL server.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie13a97ad6830553cbd099189dfb82d6f02ce82fe
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Zhou Zheng Sheng <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Eduardo <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Royce Lv <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Shu Ming <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to