Skip to content

Graduate Alertmanager API and sharding from experimental#7315

Open
friedrichg wants to merge 2 commits intomasterfrom
graduate-alertmanager
Open

Graduate Alertmanager API and sharding from experimental#7315
friedrichg wants to merge 2 commits intomasterfrom
graduate-alertmanager

Conversation

@friedrichg
Copy link
Member

@friedrichg friedrichg commented Mar 3, 2026

Summary

  • New flag is -alertmanager.enable-api. The old flag is deprecated but still valid
  • Remove WarnExperimentalUse("Alertmanager sharding") call
  • Remove Alertmanager API and sharding entries from experimental features list in v1-guarantees.md

Test plan

  • Verify alertmanager package compiles without errors
  • Verify v1-guarantees.md renders correctly

Remove experimental markers for Alertmanager API and sharding:
- New flag is -alertmanager.enable-api. The old flag is deprecated but still valid
- Remove WarnExperimentalUse("Alertmanager sharding") call
- Remove both items from experimental features list in v1-guarantees.md

The flag name itself is kept unchanged for backwards compatibility.

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@friedrichg friedrichg force-pushed the graduate-alertmanager branch from eef39cb to 10c6d13 Compare March 3, 2026 01:28
@pull-request-size pull-request-size bot added size/M and removed size/S labels Mar 3, 2026
Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@friedrichg friedrichg force-pushed the graduate-alertmanager branch from 3200909 to 1c578a4 Compare March 3, 2026 02:18
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/alertmanager lgtm This PR has been approved by a maintainer size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants