anobre/openshift-resource-governance
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix CPU and Memory summary calculation

Browse Source 
- Change from sum() to current value (last point) for accurate usage
- CPU and Memory should show current usage, not sum of all data points
- Fixes issue where memory usage was incorrectly showing 800+ MB
- Now shows realistic current resource consumption values
This commit is contained in:
Anderson
2025-10-03 20:29:04 -03:00
parent a4cf3d65bc
commit 605622f7db
1 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
app/services/historical_analysis.py
View File

@@ -1475,9 +1475,10 @@ class HistoricalAnalysisService:
datetime.utcnow())
if data and len(data) > 0:
# Sum all pod values for the workload
total_cpu = sum(self._safe_float(point[1]) for point in data if point[1] != 'NaN')
return total_cpu
# Get current value (last point) for the workload
# For CPU, we want the current rate, not sum of all points
current_cpu = self._safe_float(data[-1][1]) if data[-1][1] != 'NaN' else 0
return current_cpu
return 0.0
@@ -1514,9 +1515,10 @@ class HistoricalAnalysisService:
datetime.utcnow())
if data and len(data) > 0:
# Sum all pod values for the workload
total_memory = sum(self._safe_float(point[1]) for point in data if point[1] != 'NaN')
return total_memory
# Get current value (last point) for the workload
# For memory, we want the current usage, not sum of all points
current_memory = self._safe_float(data[-1][1]) if data[-1][1] != 'NaN' else 0
return current_memory
return 0.0