diff --git a/server/src/com/cloud/storage/StorageManagerImpl.java b/server/src/com/cloud/storage/StorageManagerImpl.java index 59c83edda15..9321585725c 100755 --- a/server/src/com/cloud/storage/StorageManagerImpl.java +++ b/server/src/com/cloud/storage/StorageManagerImpl.java @@ -1789,9 +1789,9 @@ public class StorageManagerImpl implements StorageManager, StorageService, Manag } return new Pair(hostId, answers.toArray(new Answer[answers.size()])); } catch (AgentUnavailableException e) { - s_logger.debug("Unable to send storage pool command to " + pool, e); + s_logger.debug("Unable to send storage pool command to " + pool + " via " + hostId, e); } catch (OperationTimedoutException e) { - s_logger.debug("Unable to send storage pool command to " + pool, e); + s_logger.debug("Unable to send storage pool command to " + pool + " via " + hostId, e); } } diff --git a/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java b/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java index 430fe3ed012..72d2fb5aa06 100755 --- a/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java +++ b/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java @@ -216,6 +216,10 @@ public class SnapshotManagerImpl implements SnapshotManager, SnapshotService, Ma if (result.second().getResult()) { return result.second(); } + if (s_logger.isDebugEnabled()) { + s_logger.debug("The result for " + cmd.getClass().getName() + " is " + result.second().getDetails() + " through " + result.first()); + } + hostIdsToAvoid.add(result.first()); } catch (StorageUnavailableException e1) { s_logger.warn("Storage unavailable ", e1); return null; @@ -225,7 +229,11 @@ public class SnapshotManagerImpl implements SnapshotManager, SnapshotService, Ma Thread.sleep(_pauseInterval * 1000); } catch (InterruptedException e) { } + + s_logger.debug("Retrying..."); } + + s_logger.warn("After " + _totalRetries + " retries, the command " + cmd.getClass().getName() + " did not succeed."); return null; }