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

POWERMON-306: fix wrong Prometheus rule reference for OTHER Power Consumption #410

Merged

Conversation

vprashar2929
Copy link
Collaborator

This commit fixes the issue where the wrong Prometheus rule was referenced for displaying OTHER Power Consumption(W) by Pods in the dashboard

@vprashar2929
Copy link
Collaborator Author

Verified the same on OpenShift

Screenshot 2024-07-16 at 6 14 04 PM

@vprashar2929 vprashar2929 requested a review from sthaha July 16, 2024 13:35
Copy link
Collaborator

@sthaha sthaha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @vprashar2929 . Requesting changes to

  • refer to the original issue being fixed in the commit msg
  • remove backticks from commit message

…sumption

This commit fixes the issue where the wrong Prometheus rule was referenced
for displaying `OTHER Power Consumption(W) by Pods` in the dashboards

Signed-off-by: vprashar2929 <[email protected]>
@vprashar2929 vprashar2929 changed the title fix(dashboard): correct prometheus rule for OTHER Power Consumption by Pods POWERMON-306: fix wrong Prometheus rule reference for OTHER Power Consumption Jul 17, 2024
@sthaha sthaha merged commit 780d860 into sustainable-computing-io:v1alpha1 Jul 22, 2024
11 checks passed
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.

2 participants