Change in ovirt-engine[master]: engine: Fixed validation for gluster peer detach

ofrenkel at redhat.com ofrenkel at redhat.com
Mon Jun 11 11:54:53 UTC 2012


Omer Frenkel has submitted this change and it was merged.

Change subject: engine: Fixed validation for gluster peer detach
......................................................................


engine: Fixed validation for gluster peer detach

The gluster peer detach should not be performed if the host being
removed is the only one left in the cluster. This was being checked
by comparing number of hosts across all clusters, whereas correct
way is to check number of hosts in current cluster. Fixed the same.

Change-Id: Id60b4f3387665f69e97fd9c0774fa3f5aa9a0004
Signed-off-by: Shireesh Anjal <sanjal at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVdsCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Omer Frenkel: Looks good to me, approved
  Dhandapani Gopal: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id60b4f3387665f69e97fd9c0774fa3f5aa9a0004
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shireesh Anjal <sanjal at redhat.com>
Gerrit-Reviewer: Dhandapani Gopal <dgopal at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal at redhat.com>



More information about the Engine-commits mailing list