Timothy Asir has posted comments on this change. Change subject: gluster: fix vg create failed first time during brick create ......................................................................
Patch Set 8: (1 comment) https://gerrit.ovirt.org/#/c/41392/8//COMMIT_MSG Commit Message: Line 10: blivet to create a vg. This issue not occured if both pv and vg Line 11: are created using blivet itself. Line 12: But unfortunately blivet has few issues while creating a vg. Line 13: Its able to create a vg with a pesize of minimum 1mb only. Line 14: Also the vdsm lvm module has some issue while creating a vg. > I sent a fix patch to vdsm: https://gerrit.ovirt.org/#/c/42070/. vgcreate in vdsm.storage.lvm module addes prefix, check and initialize the given pv devices forcefully. It won't be applicable to us. So we have sent a patch to make it optional so that it can be used by any module. Looks like it will take some more time to get merge. Could you please consider this patch uses lvm command. Line 15: So this path uses lvm vgcreate command to create the vg Line 16: as a workaround. Line 17: Line 18: Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1226186 -- 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: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Timothy Asir <[email protected]> Gerrit-Reviewer: Bala.FA <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Darshan N <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Ramesh N <[email protected]> Gerrit-Reviewer: Sahina Bose <[email protected]> Gerrit-Reviewer: Shubhendu Tripathi <[email protected]> Gerrit-Reviewer: Timothy Asir <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
