Change in vdsm[master]: CannotRemoveLogicalVolume is raised with arguments.

shaohef at linux.vnet.ibm.com shaohef at linux.vnet.ibm.com
Tue Oct 30 05:07:12 UTC 2012


ShaoHe Feng has posted comments on this change.

Change subject: CannotRemoveLogicalVolume is raised with arguments.
......................................................................


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

(1 inline comment)

....................................................
File vdsm/storage/storage_exception.py
Line 1305: 
Line 1306: 
Line 1307: class CannotRemoveLogicalVolume(StorageException):
Line 1308:     def __init__(self, vgName, lvNames):
Line 1309:         self.value("vg: %s lvs: %s", vgName, ", ".join(lvNames))
self.value("vg: %s lvs: %s" % (vgName, ", ".join(lvNames)))?
Line 1310:         self.vg = vgName
Line 1311:         self.lvs = lvNames
Line 1312:     code = 551
Line 1313:     message = "Failed to remove some LVs."


--
To view, visit http://gerrit.ovirt.org/8808
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ied29c1dddf4e9d6cf890cb66d362acf9eed99ba6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: ShaoHe Feng <shaohef at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list