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 <san...@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 <san...@redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Sahina Bose <sab...@redhat.com> Gerrit-Reviewer: Shireesh Anjal <san...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
