Skip to content

feat(observability): expose Prometheus metrics and wire ServiceMonitor#6807

Draft
clonder wants to merge 5 commits into
mainfrom
feat/observability-stack
Draft

feat(observability): expose Prometheus metrics and wire ServiceMonitor#6807
clonder wants to merge 5 commits into
mainfrom
feat/observability-stack

Conversation

@clonder

@clonder clonder commented Jun 30, 2026

Copy link
Copy Markdown
Contributor
  • add micrometer-registry-prometheus dependency to backend so /actuator/prometheus endpoint exists
  • enable the endpoint via backendExtraArgs in values.yaml
  • add app/component labels to loculus-backend-service so the ServiceMonitor selector can find it
  • add ServiceMonitor for Prometheus Operator to scrape the backend every 15s (requires release=kube-prometheus-stack label to be picked up by the existing kube-prometheus-stack instance)

🚀 Preview: https://feat-observability-stack.loculus.org

- Add micrometer-registry-prometheus dependency to backend so
  /actuator/prometheus endpoint exists
- Enable the endpoint via backendExtraArgs in values.yaml
- Add app/component labels to loculus-backend-service so the
  ServiceMonitor selector can find it
- Add ServiceMonitor for Prometheus Operator to scrape the backend
  every 15s (requires release=kube-prometheus-stack label to be
  picked up by the existing kube-prometheus-stack instance)
@clonder clonder requested a review from chaoran-chen June 30, 2026 13:02
@claude claude Bot added backend related to the loculus backend component deployment Code changes targetting the deployment infrastructure labels Jun 30, 2026
e2e CI cluster does not have Prometheus Operator CRDs installed,
so creating a ServiceMonitor resource fails with 'no matches for kind'.

Add enableServiceMonitor boolean (default false) and only render the
ServiceMonitor when it is true. Set it to true in values_preview_server.yaml
so production picks it up via kube-prometheus-stack.
@chaoran-chen chaoran-chen added the preview Triggers a deployment to argocd label Jun 30, 2026
@clonder clonder marked this pull request as draft June 30, 2026 13:52
@chaoran-chen chaoran-chen removed their request for review June 30, 2026 13:53
@corneliusroemer

corneliusroemer commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

I think this relates to issue:

Comment thread kubernetes/loculus/values.yaml Outdated
@corneliusroemer

Copy link
Copy Markdown
Contributor

Nice, so easy to add!

Google Chrome 2026-06-30 19 43 50

URL for above graph

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend related to the loculus backend component deployment Code changes targetting the deployment infrastructure preview Triggers a deployment to argocd

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants