Skip to content

Prepare 0.34.1 - #5556

Open
Spaceman1701 wants to merge 3 commits into
prometheus:release-0.34from
Spaceman1701:prepare-0.34.1
Open

Prepare 0.34.1#5556
Spaceman1701 wants to merge 3 commits into
prometheus:release-0.34from
Spaceman1701:prepare-0.34.1

Conversation

@Spaceman1701

Copy link
Copy Markdown
Contributor

Prepare 0.34.1 with the inhibitor fixes from #5542 and dependency updates from #5468

@Spaceman1701
Spaceman1701 requested a review from a team as a code owner September 11, 2026 10:17
Mostafa-Yehia and others added 3 commits September 11, 2026 06:22
…s#5468)

chore(deps): bump golang.org/x/mod to fix CVEs

Signed-off-by: Mostafa-Yehia <mostafa.yehia.iti@gmail.com>
Signed-off-by: Ethan Hunter <ehunter@hudson-trading.com>
This fix is based on HRT's inhibitor implementation. It updates the index so that each equal-labels key holds a `FingerprintSet` instead of a `Fingerprint` so that the index stores _every_ matching source alert instead of _one_ matching source alert.

This allows the inhibitor to search through all the alerts in the index for the set of equal-labels to select the best candidate. Worst case, this does degrade inhibitor performance to `O(n)`, but in practice it should be similar to today (since the first element of the set is most likely a viable candidate).

```release-notes
[BUGFIX] fix various issues with the inhibitor improperly skipping inhibits when the indexed source alert is resolved but other alerts are present.
```

---------

Signed-off-by: Ethan Hunter <ehunter@hudson-trading.com>
Signed-off-by: Ethan Hunter <ehunter@hudson-trading.com>
Comment thread CHANGELOG.md

## 0.34.1 / 2026-09-11

* [BUGFIX] fix various issues with the inhibitor improperly skipping inhibits when the indexed source alert is resolved but other alerts are present.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • We uppercase the start of these.
  • link the PR number at the end.
  • Optionally have the inhibit component at the start.

Could we also make it a bit more user-facing?

[BUGFIX] inhibit: Fix alerts not being muted when a source alert resolves while other matching source alerts are still firing. #5542

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants