Nir Soffer has submitted this change and it was merged.

Change subject: gluster: Don't fail connect server when getting volume info
......................................................................


gluster: Don't fail connect server when getting volume info

When connecting to gluster server, we check volume info in order
to validate replica count. However, if gluster server is not
available, we get an exception and fail the operation. This patch
catches the thrown exception and logs a warning message indicating
that we got error when tried to connect to gluster server.

Change-Id: I0e1835b7de4c5c4c5c4616d3c36f15c1f91a01dc
Bug-Url: https://bugzilla.redhat.com/1303977
Signed-off-by: Ala Hino <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/53785
Continuous-Integration: Jenkins CI
Reviewed-by: Freddy Rolland <[email protected]>
Reviewed-by: Sahina Bose <[email protected]>
Reviewed-by: Nir Soffer <[email protected]>
---
M tests/storageServerTests.py
M vdsm/storage/storageServer.py
2 files changed, 41 insertions(+), 3 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0e1835b7de4c5c4c5c4616d3c36f15c1f91a01dc
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ala Hino <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Ala Hino <[email protected]>
Gerrit-Reviewer: Freddy Rolland <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to