Yevgeny Zaspitsky has uploaded a new change for review.

Change subject: engine: Add compensation for storage pool in 
RemoveStoragePoolCommand
......................................................................

engine: Add compensation for storage pool in RemoveStoragePoolCommand

Change-Id: I491966e1901f5e792ebb57db65e1e076bebaddaf
Bug-Url: https://bugzilla.redhat.com/1087577
Signed-off-by: Yevgeny Zaspitsky <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStoragePoolCommand.java
1 file changed, 7 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/71/28671/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStoragePoolCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStoragePoolCommand.java
index 7eb78bd..d5dd478 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStoragePoolCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStoragePoolCommand.java
@@ -85,11 +85,17 @@
             }
         }
 
-        getStoragePoolDAO().remove(getStoragePool().getId());
+        removeDataCenter();
         MacPoolManager.getInstance().freeMacs(macsToRemove);
         setSucceeded(true);
     }
 
+    private void removeDataCenter() {
+        getCompensationContext().snapshotEntity(getStoragePool());
+        getStoragePoolDAO().remove(getStoragePool().getId());
+        getCompensationContext().stateChanged();
+    }
+
     private void removeNetworks() {
         final List<Network> networks = 
getNetworkDAO().getAllForDataCenter(getStoragePoolId());
         for (Network network : networks) {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I491966e1901f5e792ebb57db65e1e076bebaddaf
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to