Francesco Romani has submitted this change and it was merged.

Change subject: gluster: Warn instead of failing for unsupported gluster 
replica modes
......................................................................


gluster: Warn instead of failing for unsupported gluster replica modes

A previous patch enforced the use of only supported gluster replication
modes.  If an unsupported mode was requested, the connection attempt
would fail.  Unfortunately, this was a bit heavy-handed and has resulted
in breaking working environments during upgrade. To lessen the impact on
users, instead of failing, log a warning. A related patch to
ovirt-engine will warn users about the unsupported replication mode in
the UI.

Change-Id: I359e2405153f45b6ae303e00eb0e04a5ae14cb99
Signed-off-by: Ala Hino <ah...@redhat.com>
Bug-Url: https://bugzilla.redhat.com/1286565
Reviewed-on: https://gerrit.ovirt.org/50119
Reviewed-by: Nir Soffer <nsof...@redhat.com>
Reviewed-by: Sahina Bose <sab...@redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-on: https://gerrit.ovirt.org/50258
Reviewed-by: Francesco Romani <from...@redhat.com>
---
M tests/storageServerTests.py
M vdsm/storage/storageServer.py
2 files changed, 6 insertions(+), 9 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I359e2405153f45b6ae303e00eb0e04a5ae14cb99
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Ala Hino <ah...@redhat.com>
Gerrit-Reviewer: Ala Hino <ah...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@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