Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

discovery/kubernetes: add container image as metadata #11034

Merged
merged 2 commits into from
Jul 19, 2022

Conversation

rfratto
Copy link
Member

@rfratto rfratto commented Jul 18, 2022

This commits adds __meta_kubernetes_pod_container_image as a new metadata label. This can be used to alert on mismatched versions of targets who don't have a build_info metric, as well as injecting it into log lines for other consumers of discovery/kubernetes (e.g., Promtail).

A Grafana Agent user was asking for this metadata to be available for feature parity with Datadog.

Signed-off-by: Robert Fratto robertfratto@gmail.com

This commits adds a __meta_kubernetes_pod_container_image as a new
metadata label. This can be used to alert on mismatched versions of
targets who don't have a build_info metric, as well as injecting it into
log lines for other consumers of discovery/kubernetes (e.g., Promtail).

Signed-off-by: Robert Fratto <robertfratto@gmail.com>
@rfratto rfratto requested a review from brancz as a code owner July 18, 2022 12:36
Signed-off-by: Robert Fratto <robertfratto@gmail.com>
@brancz
Copy link
Member

brancz commented Jul 19, 2022

Nice!

@brancz brancz merged commit bb72dd8 into prometheus:main Jul 19, 2022
@rfratto rfratto deleted the k8s-container-image-metadata branch July 19, 2022 12:44
valyala added a commit to VictoriaMetrics/VictoriaMetrics that referenced this pull request Aug 14, 2022
…iner_image` label in the same way as Prometheus 2.38 does

See prometheus/prometheus#11034
valyala added a commit to VictoriaMetrics/VictoriaMetrics that referenced this pull request Aug 14, 2022
…iner_image` label in the same way as Prometheus 2.38 does

See prometheus/prometheus#11034
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants