Yaniv Bronhaim has posted comments on this change.
Change subject: Adding isSpm parameter to getAllTasksList\Status
......................................................................
Patch Set 2: (1 inline comment)
....................................................
File vdsm/storage/hsm.py
Line 310: # If no pool exists, means not SPM
Line 311: if len(pools['poollist']) == 0:
Line 312: raise Exception('Not SPM')
Line 313: for pool in pools['poollist']:
Line 314: self.validateSPM(pool)
Check SPM is always on storage pool. Only if dc is connected to storage pool we
can have host in SPM state. Currently we can have only 1 storage pool
connected. This new function checks the connected storage pool uuid and checks
if the current host is SPM for this pool, if there is no pool connected or the
host is not the SPM for this pool, it throws exception
Line 315:
Line 316: def validateNotSPM(self, spUUID):
Line 317: pool = self.getPool(spUUID)
Line 318: if pool.spmRole != sp.SPM_FREE:
--
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: 2
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: 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
