Change in vdsm[master]: gluster: Patch adds geo-replication create and delete verbs.

barumuga at redhat.com barumuga at redhat.com
Wed Mar 11 07:54:37 UTC 2015


Bala.FA has posted comments on this change.

Change subject: gluster: Patch adds geo-replication create and delete verbs.
......................................................................


Patch Set 26: Code-Review-1

(2 comments)

https://gerrit.ovirt.org/#/c/17644/26/vdsm/gluster/cli.py
File vdsm/gluster/cli.py:

Line 1342: userName
have userName as remoteUserName


Line 1361: userName
1. same as above.

2. applicable to api.py and other places too

3. in what version of glusterfs is this feature available?  you should set correct version in the spec file as well


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If8c979a89ce11a1622819c474b59dcf088733594
Gerrit-PatchSet: 26
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron 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: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list