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

Replace failing Prometheus query #161

Merged
merged 1 commit into from
Jan 15, 2025
Merged

Replace failing Prometheus query #161

merged 1 commit into from
Jan 15, 2025

Conversation

jrauh01
Copy link
Collaborator

@jrauh01 jrauh01 commented Jan 14, 2025

Problem: The daemon failed on start with Prometheus feature enabled.

Solution:
Replacing the query
sum by (instance, namespace, pod) (container_memory_usage_bytes) / on () group_left () label_replace(node_memory_MemTotal_bytes, "instance", "$1", "node", "(.*)")

by the following query
sum by (instance, namespace, pod) (container_memory_usage_bytes) / on (instance) group_left () label_replace(sum by (node) (node_memory_MemTotal_bytes), "instance", "$1", "node", "(.*)")

Fixes #141

@jrauh01 jrauh01 self-assigned this Jan 14, 2025
@cla-bot cla-bot bot added the cla/signed CLA is signed by all contributors of a PR label Jan 14, 2025
@jrauh01 jrauh01 requested a review from lippserd January 14, 2025 14:23
@lippserd lippserd merged commit 16a66d6 into main Jan 15, 2025
7 checks passed
@lippserd lippserd deleted the fix-prometheus-query branch January 15, 2025 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla/signed CLA is signed by all contributors of a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error Enabling Prometheus Integration
2 participants