Skip to content

Conversation

alvinlin123
Copy link
Contributor

@alvinlin123 alvinlin123 commented Oct 12, 2022

Signed-off-by: Alvin Lin [email protected]

What this PR does:

Add status label to cortex_alertmanager_alerts_received_total for better troubleshooting experience. status label maybe either active or resolved

Which issue(s) this PR fixes:
N/A

Checklist

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

Signed-off-by: Alvin Lin <[email protected]>
@alvinlin123 alvinlin123 marked this pull request as ready for review October 12, 2022 10:14
@alvinlin123 alvinlin123 enabled auto-merge (squash) November 5, 2022 00:41
@alvinlin123 alvinlin123 merged commit 997e4af into cortexproject:master Nov 5, 2022
t00350320 pushed a commit to t00350320/cortex that referenced this pull request Nov 15, 2022
…ric (cortexproject#4907)

* grab 'status' label for cortex_alertmanager_alerts_received_total metric

Signed-off-by: Alvin Lin <[email protected]>

* Update CHANGELOG

Signed-off-by: Alvin Lin <[email protected]>

Signed-off-by: Alvin Lin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants