Change in vdsm[master]: [WIP] Added getStorageDevices verb.

barumuga at redhat.com barumuga at redhat.com
Tue Apr 24 13:40:22 UTC 2012


Bala.FA has posted comments on this change.

Change subject: [WIP] Added getStorageDevices verb.
......................................................................


Patch Set 1: (2 inline comments)

It would be good to have getDeviceList (or something else) to populate these values with ignoring unwanted keys by respective managers.

Regarding non-working getDeviceList, could you tell me how I can run this verb?  I ran 'vdsClient localhost getDeviceList' returns empty list for me.

....................................................
File vdsm/storage_device_utils.py
Line 38:         return os.path.normpath(name)
I will check those modules

Line 92: 
I was trying to use the module, but relationship of pv/vg/lv are not stored in.  I will check it once again and get back to you exactly

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2cb217321a7a8dfcd1b507c7cba2888f08612207
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>


More information about the vdsm-patches mailing list