Change in ovirt-engine[master]: engine: Fixed validation when moving host to gluster cluster

kmayilsa at redhat.com kmayilsa at redhat.com
Thu Jun 27 11:02:35 UTC 2013


Kanagaraj M has submitted this change and it was merged.

Change subject: engine: Fixed validation when moving host to gluster cluster
......................................................................


engine: Fixed validation when moving host to gluster cluster

When moving a host to a gluster cluster, the host can be
added to target cluster only if target cluster is empty
or if there's another up server in target cluster
to perform the gluster command to add to gluster cluster.

The current check was causing NPE while performing
the AddGlusterServer command when there's no up server.
Fixing this with this patch.

Change-Id: Id2f2fbbc5da529932da967fc67ea8b7e34459fbd
Bug-Url: https://bugzilla.redhat.com/976896
Signed-off-by: Sahina Bose <sabose at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java
1 file changed, 18 insertions(+), 4 deletions(-)

Approvals:
  Kanagaraj M: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id2f2fbbc5da529932da967fc67ea8b7e34459fbd
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <sabose 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: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list