ofri masad has uploaded a new change for review.

Change subject: core: RunVmOnceCommand: don't override compensate (#856540)
......................................................................

core: RunVmOnceCommand: don't override compensate (#856540)

https://bugzilla.redhat.com/856540

This command overrides compensate() in order to call rollbackQuota()
which prevents this command from compensating.
Moreover, compensate() calls rollbackQuota() anyway, so this overriding
seems redundant in any event.

Change-Id: I0f26c4e56e4b74094ef918ab66e8722e8b8faaa1
Signed-off-by: Ofri Masad <oma...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmOnceCommand.java
1 file changed, 0 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/50/7950/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmOnceCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmOnceCommand.java
index f3c91c0..3e63b8e 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmOnceCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmOnceCommand.java
@@ -94,8 +94,4 @@
         QuotaManager.getInstance().decreaseStorageQuota(getStoragePool(), 
getStorageQuotaListParameters());
     }
 
-    @Override
-    protected void compensate() {
-        rollbackQuota();
-    }
 }


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

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

Reply via email to