Change in vdsm[master]: gluster: storage devices list provides device size in Mib

tjeyasin at redhat.com tjeyasin at redhat.com
Fri Mar 13 11:08:51 UTC 2015


Timothy Asir has posted comments on this change.

Change subject: gluster: storage devices list provides device size in Mib
......................................................................


Patch Set 2:

(2 comments)

https://gerrit.ovirt.org/#/c/38707/2//COMMIT_MSG
Commit Message:

Line 3: AuthorDate: 2015-03-13 14:32:38 +0530
Line 4: Commit:     Timothy Asir Jeyasingh <tjeyasin at redhat.com>
Line 5: CommitDate: 2015-03-13 15:53:42 +0530
Line 6: 
Line 7: gluster: storage devices list provides device size in Mib
> MiB,KiB,GiB: Bytes are written with upper case B.
Done
Line 8: 
Line 9: Currently it returns unit size along with device size in
Line 10: Kib/Mib/Gib.. based on the size. This patch fixes this issue
Line 11: to return the values in Mib.


https://gerrit.ovirt.org/#/c/38707/2/vdsm/gluster/storagedev.py
File vdsm/gluster/storagedev.py:

Line 23: 
Line 24: 
Line 25: def _getDeviceDict(device, createBrick=False):
Line 26:     info = {'name': device.name,
Line 27:             'size': '',
> it would be clearer to drop this line completely. No need to place 'size' a
Done
Line 28:             'devPath': device.path,
Line 29:             'devUuid': device.uuid or '',
Line 30:             'bus': device.bus or '',
Line 31:             'model': '',


-- 
To view, visit https://gerrit.ovirt.org/38707
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie56be6ff4e8ed5183f6c2111b007302473593361
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list