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(a)redhat.com>
Signed-off-by: Sahina Bose <sabose(a)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(a)redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Selvasundaram <sesubram(a)redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>