Change in vdsm[master]: gluster: fix vg create failed first time during brick create

danken at redhat.com danken at redhat.com
Sun Jun 14 18:46:54 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: gluster: fix vg create failed first time during brick create
......................................................................


gluster: fix vg create failed first time during brick create

Currently brick create uses vdsm lvm module for pv create and
blivet to create a vg. This issue not occured if both pv and vg
are created using blivet itself.
But unfortunately blivet has few issues while creating a vg.
Its able to create a vg with a pesize of minimum 1mb only.
Also the vdsm lvm module has some issue while creating a vg.
So this path uses lvm vgcreate command to create the vg
as a workaround.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1226186
Change-Id: I6ca89347b537e0ab6cffc2b1293ba9b5a32c366d
Signed-off-by: Timothy Asir Jeyasingh <tjeyasin at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/41392
Continuous-Integration: Jenkins CI
Reviewed-by: Bala.FA <barumuga at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/gluster/exception.py
M vdsm/gluster/storagedev.py
2 files changed, 28 insertions(+), 5 deletions(-)

Approvals:
  Timothy Asir: Verified
  Bala.FA: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6ca89347b537e0ab6cffc2b1293ba9b5a32c366d
Gerrit-PatchSet: 9
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: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list