From e5478457ee17954f9325ad5f540ab22303a579ec Mon Sep 17 00:00:00 2001
From: Edison Su <edison@cloud.com>
Date: Thu, 4 Aug 2011 11:53:49 -0700
Subject: [PATCH] don't put cpu/mem into reserved capacity if failed to
 start/stop vm

---
 server/src/com/cloud/capacity/CapacityManagerImpl.java | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/server/src/com/cloud/capacity/CapacityManagerImpl.java b/server/src/com/cloud/capacity/CapacityManagerImpl.java
index 498b5e888af..5f1cb9a0439 100755
--- a/server/src/com/cloud/capacity/CapacityManagerImpl.java
+++ b/server/src/com/cloud/capacity/CapacityManagerImpl.java
@@ -540,7 +540,7 @@ public class CapacityManagerImpl implements CapacityManager, StateListener<State
             } else if (event == Event.OperationRetry) {
                 releaseVmCapacity(vm, false, false, oldHostId);
             } else if (event == Event.AgentReportStopped) {
-                releaseVmCapacity(vm, false, true, oldHostId);
+                releaseVmCapacity(vm, false, false, oldHostId);
             } else if(event == Event.AgentReportMigrated) {
             	releaseVmCapacity(vm, false, false, oldHostId);
             }
@@ -554,7 +554,7 @@ public class CapacityManagerImpl implements CapacityManager, StateListener<State
             if (event == Event.AgentReportStopped) {
                 /* Release capacity from original host */
                 releaseVmCapacity(vm, false, false, vm.getLastHostId());
-                releaseVmCapacity(vm, false, true, oldHostId);
+                releaseVmCapacity(vm, false, false, oldHostId);
             } else if (event == Event.OperationFailed) {
                 /* Release from dest host */
                 releaseVmCapacity(vm, false, false, oldHostId);
@@ -562,8 +562,10 @@ public class CapacityManagerImpl implements CapacityManager, StateListener<State
                 releaseVmCapacity(vm, false, false, vm.getLastHostId());
             }
         } else if (oldState == State.Stopping) {
-            if (event == Event.AgentReportStopped || event == Event.OperationSucceeded) {
+            if (event == Event.OperationSucceeded) {
                 releaseVmCapacity(vm, false, true, oldHostId);
+            } else if (event == Event.AgentReportStopped) {
+                releaseVmCapacity(vm, false, false, oldHostId);
             } else if(event == Event.AgentReportMigrated) {
                 releaseVmCapacity(vm, false, false, oldHostId);
             }