Change in ovirt-engine[master]: engine: Disable gluster hook command

sanjal at redhat.com sanjal at redhat.com
Tue Apr 16 12:11:07 UTC 2013


Shireesh Anjal has submitted this change and it was merged.

Change subject: engine: Disable gluster hook command
......................................................................


engine: Disable gluster hook command

Added bll command to disable gluster hook on all
servers in a cluster.
This command extends from GlusterHookStatusChangeCommand
which does the work of calling the VDS command for each
of the server in the cluster.
The command succeeds if atleast one of the servers returns
success.

Refactored enable and disable hook tests to extend
from a common class.

Change-Id: I6fb3e034013d2ceb3590a61efc9bc46b1c3f378f
Signed-off-by: Selvasundaram <sesubram at redhat.com>
Signed-off-by: Sahina Bose <sabose at redhat.com>
---
A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DisableGlusterHookCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterHookStatusChangeCommand.java
A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/DisableGlusterHookCommandTest.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/EnableGlusterHookCommandTest.java
A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterHookCommandTest.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
7 files changed, 317 insertions(+), 145 deletions(-)

Approvals:
  Shireesh Anjal: Looks good to me, approved
  Sahina Bose: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6fb3e034013d2ceb3590a61efc9bc46b1c3f378f
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Selvasundaram <sesubram 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: Selvasundaram <sesubram at redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list