rhtyd commented on a change in pull request #4958:
URL: https://github.com/apache/cloudstack/pull/4958#discussion_r621993154



##########
File path: server/src/main/java/com/cloud/resource/ResourceManagerImpl.java
##########
@@ -2436,6 +2436,9 @@ public HostVO fillRoutingHostVO(final HostVO host, final 
StartupRoutingCommand s
         host.setHypervisorType(hyType);
         host.setHypervisorVersion(ssCmd.getHypervisorVersion());
         host.setGpuGroups(ssCmd.getGpuGroupDetails());
+
+        s_logger.info(String.format("Filling %s with the following information 
retrieved from the agent {\"cpus\": %s, \"cpuSpeed\": %s, \"cpuSockets\": %s, "

Review comment:
       Logging may not be necessary if it's a command that is sent from agent. 
There is log statement that should cover it already (on either mgmt  
server/agent side)




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to