Skip to content

Conversation

technowhizz
Copy link
Contributor

@technowhizz technowhizz commented Mar 1, 2023

Fixes the InstanceDown alerting rule wait time to be consistent with the alert message. The alert message says "for 5 minutes" but the rule was set to wait for 1 minute.

Setting the wait to 5m also matches the example alert provided by prometheus

Fixes the InstanceDown alerting rule wait time to be consistent with
the alert message. The alert message says "for 5 minutes" but the rule
was set to wait for 1 minute.
@technowhizz technowhizz requested a review from a team as a code owner March 1, 2023 15:52
@technowhizz technowhizz self-assigned this Mar 1, 2023
@technowhizz technowhizz enabled auto-merge (squash) March 1, 2023 15:52
@technowhizz technowhizz changed the base branch from stackhpc/yoga to stackhpc/xena March 1, 2023 15:53
@technowhizz technowhizz requested a review from jovial March 1, 2023 15:53
Copy link
Contributor

@jovial jovial left a comment

Choose a reason for hiding this comment

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

LGTM

@technowhizz technowhizz merged commit b1f78d3 into stackhpc/xena Mar 1, 2023
@technowhizz technowhizz deleted the fix/instance-down-alert branch March 1, 2023 19:19
@markgoddard markgoddard mentioned this pull request Mar 15, 2023
@markgoddard markgoddard mentioned this pull request Sep 21, 2023
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