Change in vdsm[master]: gluster: Don't fail connect server when getting volume info
nsoffer at redhat.com
nsoffer at redhat.com
Tue Mar 8 16:34:41 UTC 2016
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 <ahino at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/53785
Continuous-Integration: Jenkins CI
Reviewed-by: Freddy Rolland <frolland at redhat.com>
Reviewed-by: Sahina Bose <sabose at redhat.com>
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
---
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 <ahino at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
More information about the vdsm-patches
mailing list