Eduardo has posted comments on this change.
Change subject: sp: fix spm start when failing to produce domain ......................................................................
Patch Set 2: Code-Review-2
(1 comment)
http://gerrit.ovirt.org/#/c/25424/2/vdsm/storage/sp.py File vdsm/storage/sp.py:
Line 207: if domain.getDomainRole() == sd.REGULAR_DOMAIN: Line 208: continue Line 209: Line 210: self._backend.setDomainRegularRole(domain) Line 211: except Exception: SPM start should fail if one of the domains is unreacheable by the host designated of the SPM.
Allowing this behaviour can lead, among others, to a situation when extend request of VMs using the missing SD are not served.
If at any point in time one of the domains is unreacheable by the host, spm property should be lost.
The missing domain is reported by SPM and should be used by engine to find another host to be SPM or to change the pool definition accordingly. Line 212: # log any exception, but keep going Line 213: self.log.error("Error trying to check/update domain %s role", Line 214: sdUUID, exc_info=True) Line 215: