Azure Metrics: fix getFirstMetricValue to return the first value (#648)

* Make getFirstMetricValue to return the first value This function was previously returning the last value instead of the first value. This fix changes it and returns the value as soon as reading it. Co-authored-by: Maciek Sakrejda <[email protected]>
pganalyze · Dec 4, 2024 · a32fd47 · a32fd47
1 parent cb1c854
commit a32fd47
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/input/system/azure/system.go b/input/system/azure/system.go
@@ -283,10 +283,8 @@ func GetSystemState(ctx context.Context, server *state.Server, logger *util.Logg
 
 // getFirstMetricValue gets the first data from the time series metric and returns the value
 func getFirstMetricValue(metric *azquery.Metric) (metricValue *azquery.MetricValue) {
-	for _, timeSeriesElement := range metric.TimeSeries {
-		for _, mValue := range timeSeriesElement.Data {
-			metricValue = mValue
-		}
+	if len(metric.TimeSeries) == 0 || len(metric.TimeSeries[0].Data) == 0 {
+		return
 	}
-	return
+	return metric.TimeSeries[0].Data[0]
 }