Skip to content

Upgrade alertmanager v0.32.1#7462

Open
SungJin1212 wants to merge 3 commits intocortexproject:masterfrom
SungJin1212:Upgrade-alertmanager-0.32
Open

Upgrade alertmanager v0.32.1#7462
SungJin1212 wants to merge 3 commits intocortexproject:masterfrom
SungJin1212:Upgrade-alertmanager-0.32

Conversation

@SungJin1212
Copy link
Copy Markdown
Member

@SungJin1212 SungJin1212 commented Apr 30, 2026

Upgrade the Prometheus alertmanager version to v0.32.1

How to add alertmanager ui assets since v0.32.1

cd pkg/alertmanager/ui
./update-am-ui.sh 0.32.1

Key Alertmanager Changes

Protobuf Migration

  • clusterpb.FullState migrated from github.com/gogo/protobuf to google.golang.org/protobuf

UI Embed

Alertmanager UI assets (app/dist/) are now embedded via //go:embed app/dist in ui/web.go. The dist/ entry in .gitignore must explicitly allow vendored UI assets via !vendor/**/dist/.

Silence API Model Restructure

The swagger-generated API models now split Silence into separate types:

    • Silence — base model for creating silences (POST), with all fields as required pointers (*string, *strfmt.DateTime)
    • GettableSilence — extends Silence with ID, Status, and UpdatedAt fields (GET responses)
    • PostableSilence — extends Silence with optional ID (for upsert)
  • createdBy field is now required when creating a silence via the HTTP API
  • Matchers use open_api_models.Matchers ([]*Matcher) instead of amlabels.Matchers
  • Silence state constants renamed: SilenceStateActive/ExpiredSilenceStatusStateActive/SilenceStatusStateExpired

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]
  • docs/configuration/v1-guarantees.md updated if this PR introduces experimental flags

@SungJin1212 SungJin1212 marked this pull request as draft April 30, 2026 06:08
@dosubot dosubot Bot added component/alertmanager dependencies Pull requests that update a dependency file labels Apr 30, 2026
@SungJin1212 SungJin1212 force-pushed the Upgrade-alertmanager-0.32 branch 4 times, most recently from b6cfba4 to be25a00 Compare April 30, 2026 08:12
@SungJin1212
Copy link
Copy Markdown
Member Author

We need a CI to inject an alertmanager ui assets to the vendor automatically.

@SungJin1212 SungJin1212 force-pushed the Upgrade-alertmanager-0.32 branch from 4cfedc0 to 5718eca Compare May 4, 2026 01:04
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
@SungJin1212 SungJin1212 force-pushed the Upgrade-alertmanager-0.32 branch from c2d455a to d7f014f Compare May 4, 2026 01:35
@SungJin1212 SungJin1212 marked this pull request as ready for review May 4, 2026 01:35
@dosubot dosubot Bot added the go Pull requests that update Go code label May 4, 2026
router := route.New().WithPrefix(am.cfg.ExternalURL.Path)

ui.Register(router, webReload, util_log.GoKitLogToSlog(log.With(am.logger, "component", "ui")))
ui.Register(router)
Copy link
Copy Markdown
Member Author

@SungJin1212 SungJin1212 May 4, 2026

Choose a reason for hiding this comment

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

We maintain our alertmanager ui (github.com/cortexproject/cortex/pkg/alertmanager/ui) whenever we update the alertmanager.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label May 4, 2026
Comment thread go.mod
golang.org/x/sync v0.20.0
golang.org/x/time v0.14.0
google.golang.org/grpc v1.79.3
google.golang.org/grpc v1.80.0
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

it's a performance improvement for cortex, let's try it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/alertmanager dependencies Pull requests that update a dependency file go Pull requests that update Go code lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants