Change in vdsm[master]: BZ#751629 - Avoid migrate master if MSD is obsolete.

danken at redhat.com danken at redhat.com
Mon Nov 7 07:30:29 UTC 2011


Dan Kenigsberg has posted comments on this change.

Change subject: BZ#751629 - Avoid migrate master if MSD is obsolete.
......................................................................


Patch Set 1: I would prefer that you didn't submit this

(1 inline comment)

....................................................
File vdsm/storage/sp.py
Line 1110:             if new_msdUUID != BLANK_POOL_UUID:
Line 1111:                 #Trying to migrate master failed to reach actual msd.
Line 1112:                 raise se.StorageDomainAccessError(sdUUID)
Line 1113:         else:
Line 1114:             if self.masterDomain.sdUUID == sdUUID:
why not use isMaster()? is the change related in any way?

p.s. I do not understand the comment below. what is "the exception" mentioned there?
Line 1115:                 #Maybe there should be information in the exception that the UUID is
Line 1116:                 #not invalid because of its format but because it is equal to the SD. Will be less confusing.
Line 1117:                 #TODO: verify in masterMigrate().
Line 1118:                 if sdUUID == new_msdUUID:


--
To view, visit http://gerrit.usersys/1091
To unsubscribe, visit http://gerrit.usersys/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I995b86f62ee9026875b966d66f216d295aac9eba
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo Warszawski <ewarszaw at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>


More information about the vdsm-patches mailing list