Yaniv Bronhaim has posted comments on this change. Change subject: getAllTasksList\Status with 'spm_tasks' option retrieves info only if host SPM ......................................................................
Patch Set 6: (1 inline comment) .................................................... File vdsm/storage/hsm.py Line 310: raise se.IsSpm(spUUID) Line 311: Line 312: @classmethod Line 313: def getPool(cls, spUUID): Line 314: # If passed None, return first pool You told me to do it that way.. You told me to call getPool without specific UUID and change it to return the only pool because thats how it works currently (only one pool can possibly be connected). The engine won't pass the UUID as you commented on the last patchset, engine will pass only option val that get spm_tasks. so I don't understand what you're suggesting? are you suggesting something else or just saying that we can't do it in any way? Line 315: if spUUID is None: Line 316: try: Line 317: return cls.pools.popitem() Line 318: except KeyError: -- To view, visit http://gerrit.ovirt.org/12517 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4cbc11c924f0bd078749fea26d79b39c0dd48094 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Daniel P. Berrange <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Shu Ming <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
