Lei Li has posted comments on this change.
Change subject: Move and encapsulate preun section into vdsm-tool
......................................................................
Patch Set 8:
I am petty sure that I have tested the code and pass the build...
Take a look at the build error from Jenkin:
Traceback (most recent call last):
File "../tests/testrunner.py", line 274, in <module>
hackVdsmModule()
File "../tests/testrunner.py", line 252, in hackVdsmModule
sub = __import__(name, globals(), locals(), [], -1)
File
"/home/jenkins/workspace/vdsm_unit_tests_manual_gerrit/vdsm/SecureXMLRPCServer.py",
line 39, in <module>
from M2Crypto import SSL, X509
ImportError: No module named M2Crypto
Looks like it's caused by the commit: 5ea9b646f4fcd56ca48ed673810e86e26d44b523?
--
To view, visit http://gerrit.ovirt.org/4526
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4e7b5dc969dfb51e6880b9bb209a363609f5e123
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Lei Li <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Lei Li <[email protected]>
Gerrit-Reviewer: Ryan Harper <[email protected]>
Gerrit-Reviewer: ShaoHe Feng <[email protected]>
Gerrit-Reviewer: Wenyi Gao <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
