Nir Soffer has submitted this change and it was merged.

Change subject: gluster: Fixing connection when server is not one of the bricks
......................................................................


gluster: Fixing connection when server is not one of the bricks

When processing gluster connection info, volfileserver is set as provided by
the admin and, using volume info, backup-volfile-servers option is set. However,
there are use cases where gluster server and ovirt are configured differently
regarding ip addresses and domain names. If volfileserver appears in volume
info, it is removed in order to avoid mounting duplicate servers, otherwise,
mounting duplicate servers is possible.

Change-Id: Id3386b37cd407c52e1b8f38d54c236bffc143e2f
Bug-Url: https://bugzilla.redhat.com/1278880
Signed-off-by: Ala Hino <ah...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/48308
Continuous-Integration: Jenkins CI
Reviewed-by: Sahina Bose <sab...@redhat.com>
Reviewed-by: Nir Soffer <nsof...@redhat.com>
---
M tests/storageServerTests.py
M vdsm/storage/storageServer.py
2 files changed, 27 insertions(+), 1 deletion(-)

Approvals:
  Nir Soffer: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Sahina Bose: Looks good to me, but someone else must approve
  Ala Hino: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id3386b37cd407c52e1b8f38d54c236bffc143e2f
Gerrit-PatchSet: 10
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: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to