Igor Lvovsky has posted comments on this change.
Change subject: Change device ownership for VM's drives.
......................................................................
Patch Set 1: Looks good to me, but someone else must approve
(4 inline comments)
minor things
....................................................
File vdsm/clientIF.py
Line 587: res = self.irs.appropiateVolume(drive["GUID"], vmId)
appropiate->appropriate
....................................................
File vdsm/storage/hsm.py
Line 2401: def public_appropiateVolume(self, guid, thiefId):
appropiate->appropriate
Line 2403: Change owvnership of theguid device to vdsm:qemu
the guid
....................................................
File vdsm/supervdsmServer.py
Line 158: #self.log.error("Trigger event for GUID %s failed", guid,
exc_info=True)
remove log
--
To view, visit http://gerrit.ovirt.org/904
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I7859635b8abed67fe006886efd500eb9ac9f3b7c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <[email protected]>
Gerrit-Reviewer: Igor Lvovsky <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/vdsm-patches
