diff --git a/server/src/main/java/com/cloud/storage/StorageManagerImpl.java b/server/src/main/java/com/cloud/storage/StorageManagerImpl.java index 1742b9e9e0d..a21dfe06a49 100644 --- a/server/src/main/java/com/cloud/storage/StorageManagerImpl.java +++ b/server/src/main/java/com/cloud/storage/StorageManagerImpl.java @@ -1670,13 +1670,11 @@ public class StorageManagerImpl extends ManagerBase implements StorageManager, C final Answer answer = _agentMgr.easySend(hostId, modifyStoragePoolCommand); if (answer == null) { - throw new CloudRuntimeException("Unable to get an answer to the modify storage pool command" + pool.getId()); + throw new CloudRuntimeException(String.format("Unable to get an answer to the modify storage pool command %s", pool.getUuid())); } if (!answer.getResult()) { - String msg = "Unable to process ModifyStoragePoolCommand for " + poolId + " to the host" + hostId; - throw new CloudRuntimeException("Unable establish connection from storage head to storage pool " + pool.getId() + " due to " + answer.getDetails() + - pool.getId()); + throw new CloudRuntimeException(String.format("Unable to process ModifyStoragePoolCommand for pool %s on the host %s due to ", pool.getUuid(), hostId, answer.getDetails())); } assert (answer instanceof ModifyStoragePoolAnswer) : "Well, now why won't you actually return the ModifyStoragePoolAnswer when it's ModifyStoragePoolCommand? Pool=" + diff --git a/server/src/main/java/com/cloud/storage/StoragePoolAutomationImpl.java b/server/src/main/java/com/cloud/storage/StoragePoolAutomationImpl.java index 8280f23b24a..772b947128d 100644 --- a/server/src/main/java/com/cloud/storage/StoragePoolAutomationImpl.java +++ b/server/src/main/java/com/cloud/storage/StoragePoolAutomationImpl.java @@ -166,7 +166,7 @@ public class StoragePoolAutomationImpl implements StoragePoolAutomation { s_logger.debug("ModifyStoragePool false succeeded"); } if (pool.getPoolType() == Storage.StoragePoolType.DatastoreCluster) { - s_logger.debug(String.format("Started synchronising datastore cluster storage pool %s with vCenter", pool.getUuid()); + s_logger.debug(String.format("Started synchronising datastore cluster storage pool %s with vCenter", pool.getUuid())); storageManager.syncDatastoreClusterStoragePool(pool.getId(), ((ModifyStoragePoolAnswer) answer).getDatastoreClusterChildren(), host.getId()); } } @@ -331,7 +331,7 @@ public class StoragePoolAutomationImpl implements StoragePoolAutomation { s_logger.debug("ModifyStoragePool add succeeded"); } if (pool.getPoolType() == Storage.StoragePoolType.DatastoreCluster) { - s_logger.debug(String.format("Started synchronising datastore cluster storage pool %s with vCenter", pool.getUuid()); + s_logger.debug(String.format("Started synchronising datastore cluster storage pool %s with vCenter", pool.getUuid())); storageManager.syncDatastoreClusterStoragePool(pool.getId(), ((ModifyStoragePoolAnswer) answer).getDatastoreClusterChildren(), host.getId()); } }