-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Feat: Add Config Generator for Alertmanager's Mattermost Receiver #8188
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
Feat: Add Config Generator for Alertmanager's Mattermost Receiver #8188
Conversation
simonpasquier
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The WebhookURL value needs to be checked?
|
@simonpasquier I didn't see any other sanitize methods check it. I plan to implement it with CR when converting it. wdyt? |
simonpasquier
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Right, we don't do it for the other receivers so let's not do it there though I would love if the controller could prevent a bad config secret to be deployed. |
|
@simonpasquier Let me create an issue on this so we won't miss it. |
Description
Alertmanager 0.30.0 supports Mattermost receiver. This PR adds the config generator for Alertmanager to support the Mattermost integration.
Close: #8164
Type of change
CHANGE(fix or feature that would cause existing functionality to not work as expected)FEATURE(non-breaking change which adds functionality)BUGFIX(non-breaking change which fixes an issue)ENHANCEMENT(non-breaking change which improves existing functionality)NONE(if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)Verification
Unit Testing
Changelog entry