Sandro Bonazzola has posted comments on this change.
Change subject: Adding [start|stop]MonitoringDomain(). ......................................................................
Patch Set 1:
(2 comments)
.................................................... File client/vdsClient.py Line 1694: Line 1695: def startMonitoringDomain(self, args): Line 1696: sdUUID, hostID = args Line 1697: self.s.startMonitoringDomain(sdUUID, hostID) Line 1698: return 0, '' Always return 0? doesn't self.s.startMonitoringDomain(sdUUID, hostID) return the usual dict with ['status']['code'] ? Line 1699: Line 1700: def stopMonitoringDomain(self, args): Line 1701: sdUUID, = args Line 1702: self.s.stopMonitoringDomain(sdUUID)
Line 1698: return 0, '' Line 1699: Line 1700: def stopMonitoringDomain(self, args): Line 1701: sdUUID, = args Line 1702: self.s.stopMonitoringDomain(sdUUID) same here Line 1703: return 0, '' Line 1704: Line 1705: def snapshot(self, args): Line 1706: vmUUID, sdUUID, imgUUID, baseVolUUID, volUUID = args