Change in vdsm[master]: BZ#732416 fix deleteMultipleVolumes().

Max Benenson mbenenso at redhat.com
Sun Sep 18 13:00:39 UTC 2011


Max Benenson has posted comments on this change.

Change subject: BZ#732416 fix deleteMultipleVolumes().
......................................................................


Patch Set 1: Looks good to me, but someone else must approve

(2 inline comments)

....................................................
File vdsm/storage/image.py
Line 210:         if volumes:
Line 211:             try:
Line 212:                 #No, this is not a classmethod! No validations here
Line 213:                 mod = __import__(volclass.__module__, fromlist = ["deleteMultipleVolumes"])
Line 214:                 mod.deleteMultipleVolumes(sdUUID, volumes, postZero)
I prefer to validate that "mod" has deleteMultipleVolumes attribute actually, but I'm a paranoid. Maybe it's better to follow the EAFP...
Line 215:             except (se.CannotRemoveLogicalVolume, se.VolumeAccessError):
Line 216:                 #Any volume deletion failed, but we don't really care at this point
Line 217:                 self.log.warn("Problems during image %s deletion (%s). Continue...", exc_info=True)
Line 218: 


....................................................
File vdsm/storage/lvm.py
Line 786:             log.info("deactivate vg %s failed: rc %s - %s %s (ignored)" % (vgs, rc, out, err))
Line 787:         else:
Line 788:             raise se.VolumeGroupActionError("vgchange on vg(s) %s failed. %d %s %s" % (vgs, rc, out, err))
Line 789: 
Line 790: def changelv(vg, lvs, attrName, attrValue):
Need to be sure that all occurrences of _changelv were updated.
Line 791:     # Note:
Line 792:     # You may activate an activated LV without error
Line 793:     # but lvchange returns an error (RC=5) when activating rw if already rw
Line 794:     lvs = _normalizeargs(lvs)


--
To view, visit http://gerrit.usersys.redhat.com/943
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I03cd1cb2227b31b2e2d7e2e1c13cee5318ce76e2
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo Warszawski <ewarszaw at redhat.com>
Gerrit-Reviewer: Max Benenson <mbenenso at redhat.com>


More information about the vdsm-patches mailing list