Saggi Mizrahi has posted comments on this change.
Change subject: supervdsmServer is down after failed operation
......................................................................
Patch Set 10: I would prefer that you didn't submit this
(2 inline comments)
....................................................
File tests/superVdsmTests.py
Line 1: from testrunner import VdsmTestCase as TestCaseBase
Line 2: import supervdsm
Add file to Makefile
Line 3:
Line 4: class TestSuperVdsm(TestCaseBase):
Line 5:
Line 6: def setUp(self):
....................................................
File vdsm/supervdsm.py
Line 43:
Line 44: raise RuntimeError("SuperVDSM Server not found")
Line 45:
Line 46: PIDFILE = os.path.join(constants.P_VDSM_RUN, "svdsm.pid")
Line 47: TIMESTEMP = os.path.join(constants.P_VDSM_RUN, "svdsm.time")
Add this file to the whitelist in clientif.py _cleanOldFiles() so it wouldn't
get deleted
Line 48: ADDRESS = os.path.join(constants.P_VDSM_RUN, "svdsm.sock")
Line 49: SUPERVDSM = __supervdsmServerPath()
Line 50:
Line 51:
--
To view, visit http://gerrit.ovirt.org/7901
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Idad4a622b82259b777851d1b0c1b37ec8da2b01e
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Eduardo <[email protected]>
Gerrit-Reviewer: Igor Lvovsky <[email protected]>
Gerrit-Reviewer: Royce Lv <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Shu Ming <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Yeela Kaplan <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
