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

…edacore#2365) Signed-off-by: Brandon Pinske <brandon.pinske@crowdstrike.com>
bpinske · Dec 27, 2021 · 73cbe54 · 73cbe54
1 parent 8e1abee
commit 73cbe54
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -28,6 +28,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
@@ -191,7 +191,12 @@ func (s *graphiteScaler) ExecuteGrapQuery() (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
 }