Issue #368: KM module: issues with the VM class

sentrysoftware · Sep 20, 2024 · 6750adb · 6750adb
1 parent e3f9b23
commit 6750adb
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/...java/org/sentrysoftware/metricshub/hardware/sustainability/VmPowerAndEnergyEstimator.java b/...java/org/sentrysoftware/metricshub/hardware/sustainability/VmPowerAndEnergyEstimator.java
@@ -69,13 +69,16 @@ public VmPowerAndEnergyEstimator(
 	protected Double doPowerEstimation() {
 		// Get the vm power share, always >= 0.0 here
 		final double vmPowerShare = HwCollectHelper.getVmPowerShare(monitor);
+
 		// Getting the VM's power share ratio
 		final String powerSourceId = monitor.getAttribute(POWER_SOURCE_ID_ATTRIBUTE);
 		final Double totalPowerShares = totalPowerSharesByPowerSource.get(powerSourceId);
+
 		// totalPowerShares is never null here because the VM always comes with a powerShare value
 		final double powerShareRatio = totalPowerShares != null && totalPowerShares > 0.0
 			? vmPowerShare / totalPowerShares
 			: 0.0;
+
 		// Getting the power source's power consumption value
 		final Monitor powerSourceMonitor = telemetryManager.findMonitorById(powerSourceId);
 
@@ -108,6 +111,7 @@ protected Double doPowerEstimation() {
 		if (powerSourcePowerConsumption != null && powerSourcePowerConsumption >= 0.0) {
 			estimatedPower = NumberHelper.round(powerSourcePowerConsumption * powerShareRatio, 2, RoundingMode.HALF_UP);
 		}
+
 		final MetricFactory metricFactory = new MetricFactory(telemetryManager.getHostname());
 		metricFactory.collectNumberMetric(
 			monitor,