Yaniv Bronhaim has posted comments on this change. Change subject: Adding supervdsm uts ......................................................................
Patch Set 3: (4 inline comments) .................................................... Commit Message Line 4: Commit: Yaniv Bronhaim <[email protected]> Line 5: CommitDate: 2012-10-17 18:33:58 +0200 Line 6: Line 7: Adding supervdsm uts Line 8: but this is exactly what I do in this patch... Line 9: Change-Id: I08880b12db8a6ca9cc57d74d2ef2a86980a5b097 .................................................... File tests/superVdsmTests.py Line 17: self._origintimestempfile = self._proxy.timestamp Line 18: self._originsock = self._proxy.address Line 19: Line 20: # temporary values to run temporary svdsm Line 21: self._proxy.pidfile = tempfile.mkstemp()[1] I do that.. when I call to kill() in tearDown Line 22: self._proxy.timestamp = tempfile.mkstemp()[1] Line 23: self._proxy.address = tempfile.mkstemp()[1] Line 24: Line 25: def tearDown(self): .................................................... File vdsm/supervdsm.py Line 92: def __init__(self): Line 93: self._firstLaunch = True Line 94: self.pidfile = PIDFILE Line 95: self.timestamp = TIMESTAMP Line 96: self.address = ADDRESS I don't understand... what comment do you want? Line 97: Line 98: def open(self, *args, **kwargs): Line 99: return self._manager.open(*args, **kwargs) Line 100: Line 165: self._log.debug("Trying to connect to Super Vdsm") Line 166: try: Line 167: self._manager.connect() Line 168: except Exception, ex: Line 169: self._log.debug("Connect to %s failed %s", self.address, ex) I don't want to change it now. maybe next patch. this patch is here only to add uts to supervdsm, not to change its flow. Line 170: raise Line 171: self._svdsm = self._manager.instance() Line 172: Line 173: def launch(self): -- To view, visit http://gerrit.ovirt.org/8600 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I08880b12db8a6ca9cc57d74d2ef2a86980a5b097 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
