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