Federico Simoncelli has posted comments on this change.

Change subject: vm: new parameter 'protected' for vm creation
......................................................................


Patch Set 1:

(1 comment)

....................................................
File vdsm/vm.py
Line 2687:         domxml.appendGraphics()
Line 2688: 
Line 2689:         self._appendDevices(domxml)
Line 2690: 
Line 2691:         if (config.getboolean('irs', 'use_volume_leases')
Actually I wonder if we could remove use_volume_leases
Line 2692:                 or self.isProtected()):
Line 2693: 
Line 2694:             for drive in self._devices[DISK_DEVICES][:]:
Line 2695:                 if not hasattr(drive, 'volumeChain'):


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9429ead45caac1178957a33393642817db59508f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Greg Padgett <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to