Nir Soffer has posted comments on this change.

Change subject: gluster: Set mount path based on gluster volume info
......................................................................


Patch Set 8: -Code-Review

(1 comment)

One more thing - the title needs update as well.

https://gerrit.ovirt.org/#/c/48308/8//COMMIT_MSG
Commit Message:

Line 3: AuthorDate: 2015-11-09 17:56:45 +0200
Line 4: Commit:     Ala Hino <ah...@redhat.com>
Line 5: CommitDate: 2015-11-10 22:43:31 +0200
Line 6: 
Line 7: gluster: Set mount path based on gluster volume info
The title is still describing the old change. This should be something like 
"fixing connection when server is not one of the bricks"
Line 8: 
Line 9: When processing gluster connection info, volfileserver is set as 
provided by
Line 10: the admin and, using volume info, backup-volfile-servers option is 
set. However,
Line 11: there are use cases where gluster server and ovirt are configured 
differently


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id3386b37cd407c52e1b8f38d54c236bffc143e2f
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ala Hino <ah...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Ala Hino <ah...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to