Change in ovirt-engine[master]: gluster: fix remove host when host is not part of cluster

ofrenkel at redhat.com ofrenkel at redhat.com
Mon Jan 13 13:38:52 UTC 2014


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 <kmayilsa at redhat.com>
---
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 <kmayilsa at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list