Change in vdsm[ovirt-3.5-gluster]: gluster: Added an optional argument to mountBrokerSetup verb

barumuga at redhat.com barumuga at redhat.com
Sat Jun 13 06:54:40 UTC 2015


Bala.FA 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=1230353
Signed-off-by: Darshan N <dnarayan at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/41600
Reviewed-by: Sahina Bose <sabose at redhat.com>
Reviewed-by: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Reviewed-by: Bala.FA <barumuga at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-on: https://gerrit.ovirt.org/42298
Continuous-Integration: Bala.FA <barumuga at redhat.com>
---
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:
  Bala.FA: Looks good to me, approved; Passed CI tests
  Darshan N: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I732675c611221cf10f191f69990f78bd713da245
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5-gluster
Gerrit-Owner: Darshan N <dnarayan 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: automation at ovirt.org


More information about the vdsm-patches mailing list