From c5199dd328239764a73b4ef242bd4ecd6ea3f358 Mon Sep 17 00:00:00 2001
From: prachi <prachi@cloud.com>
Date: Thu, 15 Sep 2011 16:02:20 -0700
Subject: [PATCH] Bug 11457 - NullPointerException while synching VM state from
 Starting -> Running between management server and agent

Reviewed-by: Kelven and Alex

Changes:
- Added log statements about the podId during deploying a VM.
---
 .../src/com/cloud/vm/VirtualMachineManagerImpl.java   | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/server/src/com/cloud/vm/VirtualMachineManagerImpl.java b/server/src/com/cloud/vm/VirtualMachineManagerImpl.java
index bf48a4bc457..a3f27d4c0b9 100755
--- a/server/src/com/cloud/vm/VirtualMachineManagerImpl.java
+++ b/server/src/com/cloud/vm/VirtualMachineManagerImpl.java
@@ -600,11 +600,15 @@ public class VirtualMachineManagerImpl implements VirtualMachineManager, Listene
         ServiceOfferingVO offering = _offeringDao.findById(vm.getServiceOfferingId());
         VMTemplateVO template = _templateDao.findById(vm.getTemplateId());
 
+        if (s_logger.isDebugEnabled()) {
+            s_logger.debug("Trying to deploy VM, vm has dcId: "+vm.getDataCenterIdToDeployIn()+" and podId: "+vm.getPodIdToDeployIn() );
+        }
         DataCenterDeployment plan = new DataCenterDeployment(vm.getDataCenterIdToDeployIn(), vm.getPodIdToDeployIn(), null, null, null);
         if(planToDeploy != null && planToDeploy.getDataCenterId() != 0){
             if (s_logger.isDebugEnabled()) {
-                s_logger.debug("advanceStart: DeploymentPlan is provided, using that plan to deploy");
-            }
+                s_logger.debug("advanceStart: DeploymentPlan is provided, using dcId:"+planToDeploy.getDataCenterId() + ", podId: "+ planToDeploy.getPodId() 
+                        + ", clusterId: "+ planToDeploy.getClusterId() + ", hostId: "+ planToDeploy.getHostId()+ ", poolId: "+ planToDeploy.getPoolId());
+            }            
             plan = (DataCenterDeployment)planToDeploy;
         }
         HypervisorGuru hvGuru = _hvGuruMgr.getGuru(vm.getHypervisorType());
@@ -702,6 +706,9 @@ public class VirtualMachineManagerImpl implements VirtualMachineManager, Listene
                 }
 
                 try {
+                    if (s_logger.isDebugEnabled()) {
+                        s_logger.debug("VM is being started in podId: "+vm.getPodIdToDeployIn());
+                    }                    
                     _networkMgr.prepare(vmProfile, dest, ctx);
                     if (vm.getHypervisorType() != HypervisorType.BareMetal) {
                         _storageMgr.prepare(vmProfile, dest);