Omer Frenkel has submitted this change and it was merged.

Change subject: gluster: fix remove host when host is not part of cluster
......................................................................


gluster: fix remove host when host is not part of cluster

VDSM in hosts with compatitbility version 3.3(or less)
doesn't return appropriate error status(GlusterHostIsNotPartOfCluster)
during the host remove if the operation is failed because
the host which is being removed is not part of the gluster-cluster.

The Host can be present in engine but not in the actual
gluster-cluster if the host installation is failed.

This issue is now fixed by verifying the peer list and
remove the host if the host is not found in the list.

Change-Id: I785beb895f0fb72bc486055c666936073b5a5d59
Bug-Url: https://bugzilla.redhat.com/1050135
Signed-off-by: Kanagaraj M <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVdsCommand.java
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/GlusterUtil.java
3 files changed, 82 insertions(+), 48 deletions(-)

Approvals:
  Omer Frenkel: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/23152
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I785beb895f0fb72bc486055c666936073b5a5d59
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Ramesh N <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: Shubhendu Tripathi <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to