Shireesh Anjal has submitted this change and it was merged.
Change subject: gluster: Improved gluster validation in add host
......................................................................
gluster: Improved gluster validation in add host
When checking if the host is part of another gluster cluster,
- Ignore if the command (gluster peer status) fails, to handle cases
where glusterfs is not installed or glusterd is down
- Do not perform the check if the cluster in which the host is being
added is currently empty i.e. when the host being added will be the
first host of the cluster.
Change-Id: I06a2ee4621b175aa70f1ccdcf7a740d830494acf
Bug-Url:
https://bugzilla.redhat.com/956077
Signed-off-by: Shireesh Anjal <sanjal(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVdsCommandTest.java
2 files changed, 47 insertions(+), 6 deletions(-)
Approvals:
Shireesh Anjal: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/14224
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I06a2ee4621b175aa70f1ccdcf7a740d830494acf
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shireesh Anjal <sanjal(a)redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>