Ayal Baron has posted comments on this change.
Change subject: Adding the status field to getDeviceList info.
......................................................................
Patch Set 3: I would prefer that you didn't submit this
(4 inline comments)
....................................................
File vdsm/storage/lvm.py
Line 626: #The device is created with the new parameters.
comment seems redundant here
Line 642: def _initpvs_removeHolders():
missing empty line above
Line 750: unusedDevs, usedDevs, rc, out, err = _initpvsTest(devs, metadataSize)
why separate _initpvsTest out?
Line 800: pvCreateMode = True if force == "True" else False
this should be parsed at API.py
--
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: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw(a)redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com>