Skip to content

Conversation

@jovial
Copy link
Contributor

@jovial jovial commented Nov 28, 2025

redfish-exporter-seed was renamed redfish-exporter in:

782f689

This updates the rules to match and thereby reduces the sensitivity of the alert.

@jovial jovial requested a review from a team as a code owner November 28, 2025 18:11
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates Prometheus alert rules to reflect the renaming of the redfish-exporter-seed job to redfish-exporter. The changes correctly update the job name in the alert expressions. I have added one comment with a suggestion to fully align the alert rule with the intent described in the pull request (changing the severity to 'warning') and to improve its actionability by adding annotations.

redfish-exporter-seed was renamed redfish-exporter in:

782f689

This updates the rules to match. Effectively making failed redfish scrapes warnings again.
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.

4 participants