NPE fix, for test_hostha_kvm_host_fencing (#9355)

sureshanaparti · web-flow · commit 37c91abd3d4f · 2024-07-09T12:20:10.000+05:30
diff --git a/server/src/main/java/com/cloud/capacity/CapacityManagerImpl.java b/server/src/main/java/com/cloud/capacity/CapacityManagerImpl.java
@@ -718,6 +718,8 @@ public void updateCapacityForHost(final Host host, final Map<Long, ServiceOfferi
                 if (vmDetailCpu != null) {
                     //if vmDetail_cpu is not null it means it is running in a overcommited cluster.
                     cpuOvercommitRatio = Float.parseFloat(vmDetailCpu.getValue());
+                }
+                if (vmDetailRam != null) {
                     ramOvercommitRatio = Float.parseFloat(vmDetailRam.getValue());
                 }
                 ServiceOffering so = offeringsMap.get(vm.getServiceOfferingId());

Original file line number	Diff line number	Diff line change
`@@ -718,6 +718,8 @@ public void updateCapacityForHost(final Host host, final Map<Long, ServiceOfferi`
`718`	`718`	`if (vmDetailCpu != null) {`
`719`	`719`	`//if vmDetail_cpu is not null it means it is running in a overcommited cluster.`
`720`	`720`	`cpuOvercommitRatio = Float.parseFloat(vmDetailCpu.getValue());`
	`721`	`+ }`
	`722`	`+ if (vmDetailRam != null) {`
`721`	`723`	`ramOvercommitRatio = Float.parseFloat(vmDetailRam.getValue());`
`722`	`724`	`}`
`723`	`725`	`ServiceOffering so = offeringsMap.get(vm.getServiceOfferingId());`