Alissa Bonas has uploaded a new change for review.

Change subject: core: set action/type in FenceVdsManualyCommand
......................................................................

core: set action/type in FenceVdsManualyCommand

Set action and type in setActionMessageParameters so it will
be properly resolved.

Change-Id: I89bbd039d683e6b95b5ce930f3bae195414283d1
Signed-off-by: Alissa Bonas <abo...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/FenceVdsManualyCommand.java
1 file changed, 5 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/43/20143/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/FenceVdsManualyCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/FenceVdsManualyCommand.java
index 8553a23..f73db27 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/FenceVdsManualyCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/FenceVdsManualyCommand.java
@@ -62,8 +62,6 @@
     @Override
     protected boolean canDoAction() {
         boolean returnValue = true;
-        addCanDoActionMessage(VdcBllMessages.VAR__TYPE__HOST);
-        addCanDoActionMessage(VdcBllMessages.VAR__ACTION__MANUAL_FENCE);
         // check problematic vds status
         if (IsLegalStatus(_problematicVds.getStatus())) {
             if (_problematicVds.getSpmStatus() == VdsSpmStatus.SPM) {
@@ -236,4 +234,9 @@
         return Collections.singletonList(new 
PermissionSubject(getParameters().getVdsId(), VdcObjectType.VDS,
                 getActionType().getActionGroup()));
     }
+
+    protected void setActionMessageParameters() {
+        addCanDoActionMessage(VdcBllMessages.VAR__ACTION__MANUAL_FENCE);
+        addCanDoActionMessage(VdcBllMessages.VAR__TYPE__HOST);
+     }
 }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I89bbd039d683e6b95b5ce930f3bae195414283d1
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alissa Bonas <abo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to