Change in vdsm[master]: gluster: create vg with custom pesize

tjeyasin at redhat.com tjeyasin at redhat.com
Wed May 27 18:04:04 UTC 2015


Timothy Asir has posted comments on this change.

Change subject: gluster: create vg with custom pesize
......................................................................


Patch Set 2:

(1 comment)

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

Line 167:         return deviceList
Line 168: 
Line 169:     def _createVG(vgName, deviceList, stripeSize=0):
Line 170:         if stripeSize:
Line 171:             devices = ','.join([device.path for device in deviceList])
> you should add this into the spec accordingly
Sure, I will do it!
Line 172:             rc, out, err = utils.execCmd([_vgCreateCommandPath.cmd,
Line 173:                                           '-s', '%sk' % stripeSize,
Line 174:                                           vgName, devices])
Line 175: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6ca89347b537e0ab6cffc2b1293ba9b5a32c366d
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: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list