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

mixin: adapt alerts/playbooks to consider ruler query path components #1949

Merged
merged 5 commits into from
Jun 8, 2022

Conversation

ortuman
Copy link
Contributor

@ortuman ortuman commented May 30, 2022

Signed-off-by: Miguel Ángel Ortuño ortuman@gmail.com

What this PR does

Add extra alerts for ruler query path components and adapt current ones to avoid conflicting with regular query path.

Which issue(s) this PR fixes or relates to

Fixes N/A

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@ortuman ortuman requested review from stevesg and pracucci May 30, 2022 09:48
@ortuman ortuman added type/docs Improvements or additions to documentation and removed monitoring/runbooks labels May 30, 2022
@ortuman ortuman force-pushed the ortuman/mixin-ruler-query-path-alerts branch from 12ae35e to e55e286 Compare May 30, 2022 10:14
@ortuman ortuman force-pushed the ortuman/mixin-ruler-query-path-alerts branch 2 times, most recently from ed93e88 to 638ab32 Compare June 1, 2022 08:14
Copy link
Contributor

@stevesg stevesg left a comment

Choose a reason for hiding this comment

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

Looks good

…path components.

Signed-off-by: Miguel Ángel Ortuño <ortuman@gmail.com>
Signed-off-by: Miguel Ángel Ortuño <ortuman@gmail.com>
Signed-off-by: Miguel Ángel Ortuño <ortuman@gmail.com>
Signed-off-by: Miguel Ángel Ortuño <ortuman@gmail.com>
@ortuman ortuman force-pushed the ortuman/mixin-ruler-query-path-alerts branch from 0d506b7 to 083b08a Compare June 8, 2022 09:29
Signed-off-by: Miguel Ángel Ortuño <ortuman@gmail.com>
@ortuman ortuman requested a review from stevesg June 8, 2022 09:37
@pracucci
Copy link
Collaborator

pracucci commented Jun 8, 2022

Taking a final look too.

@pracucci pracucci merged commit dd1c12a into main Jun 8, 2022
@pracucci pracucci deleted the ortuman/mixin-ruler-query-path-alerts branch June 8, 2022 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
monitoring/alerts type/docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants