Skip to content

Conversation

@nutmos
Copy link
Contributor

@nutmos nutmos commented Jun 24, 2025

Description

Add global config support in the alert manager for Telegram.

Type of change

What type of changes does your code introduce to the Prometheus operator? Put an x in the box that apply.

  • 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

Using unit tests for validation

Changelog entry

- Add support for API URLs in the global configuration for the alert manager for Telegram

@nutmos nutmos requested a review from a team as a code owner June 24, 2025 13:18
nutmos and others added 2 commits June 25, 2025 19:40
Commit by Suggestion

Co-authored-by: Simon Pasquier <[email protected]>
simonpasquier
simonpasquier previously approved these changes Jun 25, 2025
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 enabled auto-merge (squash) June 25, 2025 11:43
auto-merge was automatically disabled June 25, 2025 12:15

Head branch was pushed to by a user without write access

simonpasquier
simonpasquier previously approved these changes Jun 25, 2025
@simonpasquier simonpasquier enabled auto-merge (squash) June 25, 2025 12:26
auto-merge was automatically disabled June 25, 2025 12:27

Head branch was pushed to by a user without write access

@nutmos nutmos requested a review from simonpasquier June 25, 2025 13:23
@nutmos
Copy link
Contributor Author

nutmos commented Jun 25, 2025

@simonpasquier Updated every suggestion already. Please review again.

@simonpasquier simonpasquier merged commit 115191a into prometheus-operator:main Jun 25, 2025
23 checks passed
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.

3 participants