Graphite scaler should use latest datapoint, not earliest, returned (#…

…2365) Signed-off-by: Brandon Pinske <brandon.pinske@crowdstrike.com>
kedacore · Dec 1, 2021 · 8e4f201 · 8e4f201
1 parent 1cdb887
commit 8e4f201
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -27,6 +27,8 @@
 
 ### Improvements
 
+- Graphite Scaler: use the latest datapoint returned, not the earliest (https://github.com/kedacore/keda/pull/2365)
+
 - TODO ([#XXX](https://github.com/kedacore/keda/pull/XXX))
 
 ### Breaking Changes

diff --git a/pkg/scalers/graphite_scaler.go b/pkg/scalers/graphite_scaler.go
@@ -194,7 +194,12 @@ func (s *graphiteScaler) ExecuteGrapQuery(ctx context.Context) (float64, error)
 	}
 
 	// https://graphite-api.readthedocs.io/en/latest/api.html#json
-	datapoint := result[0].Datapoints[0][0]
+	if len(result[0].Datapoints) == 0 {
+		return 0, nil
+	}
+
+	latestDatapoint := len(result[0].Datapoints) - 1
+	datapoint := result[0].Datapoints[latestDatapoint][0]
 
 	return datapoint, nil
 }