Change in ovirt-engine[master]: gluster: Improved gluster validation in add host

[sanjal at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal at redhat.com>