Skip to content

Commit

Permalink
Fix indentation in CCM ServiceMonitor (#882)
Browse files Browse the repository at this point in the history
The `honorLabels`, `authorization`, `scheme`, and `tlsConfig` fields are
part of the Endpoint API, not the Spec. This indentation error caused the
prometheus-operator to generate a scrape job configuration that specified
http instead of https (because the scheme field was "missing". This in turn
caused CCM-down alerts to fire because the scrape failed with a 400 error.
  • Loading branch information
rickardsjp authored Jun 10, 2024
1 parent 567233d commit 6604a6b
Showing 1 changed file with 8 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,12 @@ spec:
- __name__
action: keep
regex: ^(rest_client_requests_total|process_max_fds|process_open_fds)$
honorLabels: false
authorization:
credentials:
name: shoot-access-prometheus-shoot
key: token
scheme: https
tlsConfig:
insecureSkipVerify: true
honorLabels: false
authorization:
credentials:
name: shoot-access-prometheus-shoot
key: token
scheme: https
tlsConfig:
insecureSkipVerify: true
{{- end }}

0 comments on commit 6604a6b

Please sign in to comment.