rafaelweingartner commented on a change in pull request #2795: kvm: Properly
report available memory to Management Server
URL: https://github.com/apache/cloudstack/pull/2795#discussion_r222983111
##########
File path:
plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
##########
@@ -383,11 +378,11 @@ public LibvirtUtilitiesHelper
getLibvirtUtilitiesHelper() {
}
public CPUStat getCPUStat() {
- return _cpuStat;
+ return new CPUStat();
Review comment:
Is this coded on purpose? I mean, can we use it as a stateless call? At
first sight this `getCPUStat` seem to return an object property, but instead it
will always return a different object instance.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services