Change in vdsm[master]: drives config normalization.
danken at redhat.com
danken at redhat.com
Fri Dec 16 14:10:47 UTC 2011
Dan Kenigsberg has posted comments on this change.
Change subject: drives config normalization.
......................................................................
Patch Set 1: (3 inline comments)
....................................................
File vdsm/libvirtvm.py
Line 1079: #TODO: Should loop over disks running on the VM.
why?
Line 1084: for drive in drives:
appears on both branches of the condition!
....................................................
File vdsm/vm.py
Line 458: # def _initDriveList(self, drives):
why keep this?
--
To view, visit http://gerrit.ovirt.org/624
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I80bbd888acc6cccd381f04701c34f52b155e5ca7
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
More information about the vdsm-patches
mailing list