Change in vdsm[master]: Adding the status field to getDeviceList info.
abaron at redhat.com
abaron at redhat.com
Thu Jun 28 14:40:37 UTC 2012
Ayal Baron has posted comments on this change.
Change subject: Adding the status field to getDeviceList info.
......................................................................
Patch Set 4: I would prefer that you didn't submit this
(2 inline comments)
....................................................
File vdsm/storage/lvm.py
Line 626: #The device is created with the new parameters.
this comment should be in the docstring
Line 800: pvCreateMode = True if force == "True" else False
this should be done at API level (hsm.py at least)
misc.parseBool(force)
pvCreateMode should not exist
--
To view, visit http://gerrit.ovirt.org/5515
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6af6628a29bb597cf46b57a2b439e1b950bef9ab
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
More information about the vdsm-patches
mailing list