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

update web UI, Slack, and Telegram to allow silencing an acknowledged alert group #1831

Merged
merged 11 commits into from
Apr 27, 2023

Conversation

joeyorlando
Copy link
Contributor

@joeyorlando joeyorlando commented Apr 26, 2023

What this PR does

https://www.loom.com/share/1a6ef0d00c3b46ca80c120579d512dcc

Checklist

  • Unit, integration, and e2e (if applicable) tests updated (N/A)
  • Documentation added (or pr:no public docs PR label added if not required)
  • CHANGELOG.md updated (or pr:no changelog PR label added if not required)

even if alert group is acknowledged
@joeyorlando joeyorlando added the pr:no public docs Added to a PR that does not require public documentation updates label Apr 26, 2023
@joeyorlando joeyorlando requested a review from a team April 26, 2023 20:24
@mderynck
Copy link
Contributor

Does this avoid the issue of re-entering firing state and notifying someone? Also should this be done across the board for Slack and other message backends in one PR? There is some conflict with docs Acknowledged -> Silenced isn't listed as a possible transaction.

Copy link
Contributor

@iskhakov iskhakov left a comment

Choose a reason for hiding this comment

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

Cool change, let's do the same for slack/other messengers

@joeyorlando joeyorlando changed the title show silence button on alert group page even if alert group is acknowledged update web UI, Slack, and Telegram to allow silencing an acknowledged alert group Apr 27, 2023
@joeyorlando joeyorlando removed the pr:no public docs Added to a PR that does not require public documentation updates label Apr 27, 2023
Copy link
Contributor Author

Choose a reason for hiding this comment

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

not related to this change, but fixing some grammatical things I came across

@joeyorlando joeyorlando requested a review from a team April 27, 2023 14:30
@joeyorlando joeyorlando enabled auto-merge April 27, 2023 14:47
@joeyorlando joeyorlando added this pull request to the merge queue Apr 27, 2023
Merged via the queue into dev with commit 4967ce8 Apr 27, 2023
@joeyorlando joeyorlando deleted the jorlando/show-alert-group-silence-button branch April 27, 2023 14:58
brojd pushed a commit that referenced this pull request Sep 18, 2024
… alert group (#1831)

# What this PR does

https://www.loom.com/share/1a6ef0d00c3b46ca80c120579d512dcc

## Checklist

- [ ] Unit, integration, and e2e (if applicable) tests updated (N/A)
- [x] Documentation added (or `pr:no public docs` PR label added if not
required)
- [x] `CHANGELOG.md` updated (or `pr:no changelog` PR label added if not
required)
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