From fcd7f37839c778df4c7e0bb520245227e4ba32ba Mon Sep 17 00:00:00 2001 From: abhishek Date: Thu, 11 Nov 2010 17:17:40 -0800 Subject: [PATCH] bug 7037: adding more logging to the deletepool cmd status 7037: resolved fixed --- server/src/com/cloud/storage/StorageManagerImpl.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/server/src/com/cloud/storage/StorageManagerImpl.java b/server/src/com/cloud/storage/StorageManagerImpl.java index 1a9ef775f78..1e4e58f54f8 100755 --- a/server/src/com/cloud/storage/StorageManagerImpl.java +++ b/server/src/com/cloud/storage/StorageManagerImpl.java @@ -1461,10 +1461,12 @@ public class StorageManagerImpl implements StorageManager { //verify parameters StoragePoolVO sPool = _storagePoolDao.findById(id); if (sPool == null) { + s_logger.warn("Unable to find pool:"+id); throw new InvalidParameterValueException("Unable to find pool by id " + id); } if (sPool.getPoolType().equals(StoragePoolType.LVM)) { + s_logger.warn("Unable to delete local storage id:"+id); throw new InvalidParameterValueException("Unable to delete local storage id: " + id); } @@ -1484,6 +1486,7 @@ public class StorageManagerImpl implements StorageManager { Pair volumeRecords = _volsDao.getCountAndTotalByPool(id); if (volumeRecords.first() > 0) { + s_logger.warn("Cannot delete pool "+sPool.getName()+" as there are associated vols for this pool"); return false; // cannot delete as there are associated vols } // 3. Else part, remove the SR associated with the Xenserver