Dan Kenigsberg has submitted this change and it was merged. Change subject: gluster: Added an optional argument to mountBrokerSetup verb ......................................................................
gluster: Added an optional argument to mountBrokerSetup verb Added an optional argument called "partial" to mountBrokerSetup verb. This argument takes boolean value. If true only mount broker root creation is done as part of mountBrokerSetup, else along with root creation user addition and few mount broker options are also set. This is introduced because in only one of the slave node's all the operations mentioned above has to be done, this is synced to all the remaining nodes automatically. Only mount broker root creation has to be performed on all the slave nodes. Engine can call this verb with partial as false on one of the nodes and on all the other nodes verb can be called with partial set to true. Change-Id: I732675c611221cf10f191f69990f78bd713da245 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1220345 Signed-off-by: Darshan N <[email protected]> Reviewed-on: https://gerrit.ovirt.org/41600 Reviewed-by: Sahina Bose <[email protected]> Reviewed-by: Piotr Kliczewski <[email protected]> Reviewed-by: Bala.FA <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> Continuous-Integration: Jenkins CI --- M client/vdsClientGluster.py M vdsm/gluster/api.py M vdsm/gluster/apiwrapper.py M vdsm/rpc/vdsmapi-gluster-schema.json 4 files changed, 20 insertions(+), 13 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Bala.FA: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Darshan N: Verified Sahina Bose: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41600 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I732675c611221cf10f191f69990f78bd713da245 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N <[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: Sahina Bose <[email protected]> Gerrit-Reviewer: Timothy Asir <[email protected]> Gerrit-Reviewer: anmolbabu <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
