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