Skip to content

Conversation

@nutmos
Copy link
Contributor

@nutmos nutmos commented Dec 17, 2025

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

- Add the config generator for Alertmanager to support the Mattermost integration.

Copy link
Contributor

@simonpasquier simonpasquier left a 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?

@nutmos
Copy link
Contributor Author

nutmos commented Dec 18, 2025

@simonpasquier I didn't see any other sanitize methods check it. I plan to implement it with CR when converting it. wdyt?

@nutmos nutmos requested a review from simonpasquier December 18, 2025 13:31
Copy link
Contributor

@simonpasquier simonpasquier left a comment

Choose a reason for hiding this comment

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

Thanks!

@simonpasquier simonpasquier merged commit 3265109 into prometheus-operator:main Dec 18, 2025
23 checks passed
@simonpasquier
Copy link
Contributor

simonpasquier commented Dec 18, 2025

@simonpasquier I didn't see any other sanitize methods check it. I plan to implement it with CR when converting it. wdyt?

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.

@nutmos
Copy link
Contributor Author

nutmos commented Dec 19, 2025

@simonpasquier Let me create an issue on this so we won't miss it.

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.

Add Support for Alertmanager Mattermost integration

2 participants