Skip to content

Introduce new metric syn_cluster_dynamic_facts#340

Open
simu wants to merge 1 commit intomasterfrom
feat/dynfacts-metric
Open

Introduce new metric syn_cluster_dynamic_facts#340
simu wants to merge 1 commit intomasterfrom
feat/dynfacts-metric

Conversation

@simu
Copy link
Copy Markdown
Member

@simu simu commented May 7, 2026

This metric enables us to easily consume Lieutenant cluster dynamic facts through Prometheus queries (e.g. for central dashboards).

We currently don't filter out any dynamic facts, and accept that the metric may have sizeable label values.

Generally speaking, Prometheus should be able to handle bigger label values gracefully, but if the larger label values (currently primarily status.facts.kubernetesVersion) turn out to be an issue, we can always drop them with a scrape relabel config (in the ServiceMonitor for each Lieutenant instance).

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Update the documentation.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog

@simu simu added the enhancement New feature or request label May 7, 2026
This metric enables us to easily consume Lieutenant cluster dynamic
facts through Prometheus queries (e.g. for central dashboards).

We currently don't filter out any dynamic facts, and accept that the
metric may have sizeable label values.

Generally speaking, Prometheus should be able to handle bigger label
values gracefully, but if the larger label values (currently primarily
`status.facts.kubernetesVersion`) turn out to be an issue, we can always
drop them with a scrape relabel config (in the `ServiceMonitor` for each
Lieutenant instance).
Comment thread metrics/cluster_info_collector.go Outdated
@simu simu force-pushed the feat/dynfacts-metric branch from 3da0602 to 6698c0d Compare May 7, 2026 12:30
@simu simu requested a review from bastjan May 8, 2026 10:06
@simu simu marked this pull request as ready for review May 8, 2026 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants