Skip to content

Commit 25f9b0d

Browse files
committed
do not fallback to kubernetes service URL querying if aggregating metrics from all kubernetes service endpoints fail for whatever reason
see kedacore#6565 (comment) Signed-off-by: julian GUINARD <[email protected]>
1 parent 61ebd1f commit 25f9b0d

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

pkg/scalers/metrics_api_scaler.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -513,14 +513,14 @@ func (s *metricsAPIScaler) getMetricValue(ctx context.Context) (float64, error)
513513
if s.metadata.aggregateFromKubeServiceEndpoints {
514514
endpointsUrls, err := s.getEndpointsUrlsFromServiceURL(ctx, s.metadata.url)
515515
if err != nil {
516-
s.logger.Error(err, "Failed to get kubernetes endpoints urls from configured service URL. Falling back to querying url configured in metadata")
516+
return 0, fmt.Errorf("failed to get kubernetes endpoints urls from configured service URL")
517517
} else {
518518
if len(endpointsUrls) == 0 {
519-
s.logger.Error(err, "No endpoints URLs were given for the service name. Falling back to querying url configured in metadata")
519+
return 0, fmt.Errorf("no endpoints URLs were given for the service name")
520520
} else {
521521
aggregatedMetric, err := s.aggregateMetricsFromMultipleEndpoints(ctx, endpointsUrls)
522522
if err != nil {
523-
s.logger.Error(err, "No aggregated metrics could be computed from service endpoints. Falling back to querying url configured in metadata")
523+
return 0, fmt.Errorf("no aggregated metrics could be computed from service endpoints")
524524
} else {
525525
return aggregatedMetric, err
526526
}

0 commit comments

Comments
 (0)