From 9fa63cfe6173e0ea94fe657e064b47c9d005a679 Mon Sep 17 00:00:00 2001 From: Edison Su Date: Tue, 15 May 2012 14:41:47 -0700 Subject: [PATCH] CS-14609 turn on debug log, if zone is not ready to start ssvm/cpvm. "Zone host is ready, but console proxy template" means system vm template is not in "Downloaded" state on secondary storage "Primary storage is not ready" means the template is not been downloaded to primary storage, possible reasons: haven't add primary storage, or primary storage download is not finished/failed. --- .../com/cloud/consoleproxy/ConsoleProxyManagerImpl.java | 8 ++++---- .../storage/secondary/SecondaryStorageManagerImpl.java | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java b/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java index 612d2b69797..088be6296ce 100755 --- a/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java +++ b/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java @@ -1123,13 +1123,13 @@ public class ConsoleProxyManagerImpl implements ConsoleProxyManager, ConsoleProx if (l != null && l.size() > 0 && l.get(0).second().intValue() > 0) { return true; } else { - if (s_logger.isTraceEnabled()) { - s_logger.trace("Primary storage is not ready, wait until it is ready to launch console proxy"); + if (s_logger.isDebugEnabled()) { + s_logger.debug("Primary storage is not ready, wait until it is ready to launch console proxy"); } } } else { - if (s_logger.isTraceEnabled()) { - s_logger.trace("Zone host is ready, but console proxy template is not ready"); + if (s_logger.isDebugEnabled()) { + s_logger.debug("Zone host is ready, but console proxy template: " + template.getId() + " is not ready on secondary storage: " + secondaryStorageHost.getId()); } } } diff --git a/server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java b/server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java index 6accab99594..a8bc143e0c6 100755 --- a/server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java +++ b/server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java @@ -727,8 +727,8 @@ public class SecondaryStorageManagerImpl implements SecondaryStorageVmManager, V } } } else { - if (s_logger.isTraceEnabled()) { - s_logger.trace("Zone host is ready, but secondary storage vm template is not ready"); + if (s_logger.isDebugEnabled()) { + s_logger.debug("Zone host is ready, but secondary storage vm template: " + template.getId() + " is not ready on secondary storage: " + secHost.getId()); } } }