From cd9c7737d19c97c95fd536e1ecb4641d4e8651fc Mon Sep 17 00:00:00 2001 From: Nemo Date: Tue, 11 Oct 2016 12:58:02 -0400 Subject: [PATCH] Made the changes to improve logging. --- .../src/com/cloud/agent/manager/AgentManagerImpl.java | 4 ++-- .../src/com/cloud/vm/VirtualMachineManagerImpl.java | 2 +- .../src/com/cloud/hypervisor/kvm/resource/KVMHAChecker.java | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java b/engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java index aa7068af533..aaa3ba5ad48 100644 --- a/engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java +++ b/engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java @@ -1231,9 +1231,9 @@ public class AgentManagerImpl extends ManagerBase implements AgentManager, Handl } else if (cmd instanceof PingCommand) { logD = false; s_logger.debug("Ping from " + hostId + "(" + hostName + ")"); - s_logger.trace("SeqA " + attache.getId() + "-" + request.getSequence() + ": Processing " + request); + s_logger.trace("SeqA " + hostId + "-" + request.getSequence() + ": Processing " + request); } else { - s_logger.debug("SeqA " + attache.getId() + "-" + request.getSequence() + ": Processing " + request); + s_logger.debug("SeqA " + hostId + "-" + request.getSequence() + ": Processing " + request); } } diff --git a/engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java b/engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java index 9523b928ed4..ad8e9286221 100644 --- a/engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java +++ b/engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java @@ -3355,7 +3355,7 @@ public class VirtualMachineManagerImpl extends ManagerBase implements VirtualMac } if (fromHost.getClusterId().longValue() != dest.getCluster().getId()) { - s_logger.info("Source and destination host are not in same cluster, unable to migrate to host: " + dest.getHost().getId()); + s_logger.info("Source and destination host are not in same cluster, unable to migrate to host: " + dstHostId); throw new CloudRuntimeException("Source and destination host are not in same cluster, unable to migrate to host: " + dest.getHost().getId()); } diff --git a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/KVMHAChecker.java b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/KVMHAChecker.java index c2af191cae6..565673ec7bc 100644 --- a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/KVMHAChecker.java +++ b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/KVMHAChecker.java @@ -57,7 +57,7 @@ public class KVMHAChecker extends KVMHABase implements Callable { s_logger.debug("reture: " + result); s_logger.debug("parser: " + parser.getLine()); if (result == null && parser.getLine().contains("> DEAD <")) { - s_logger.debug("read heartbeat failed: " + result); + s_logger.debug("read heartbeat failed: "); results.add(false); } else { results.add(true);