Skip to content

alerts: fix duplicate series in InfoInhibitor alert#2786

Merged
simonpasquier merged 2 commits intoprometheus-operator:mainfrom
varundeepsaini:fix/infoinhibitor-duplicate-series
Mar 10, 2026
Merged

alerts: fix duplicate series in InfoInhibitor alert#2786
simonpasquier merged 2 commits intoprometheus-operator:mainfrom
varundeepsaini:fix/infoinhibitor-duplicate-series

Conversation

@varundeepsaini
Copy link
Copy Markdown
Contributor

Description

Fix the InfoInhibitor alert failing to evaluate with a "duplicate series" error in Prometheus.

When multiple info-level alerts fire in the same namespace, the current expression can produce multiple series with the same labelset after the unless on(namespace) operation, causing Prometheus to fail evaluation.

The fix uses count by (namespace) to aggregate alerts, ensuring each side of the unless operator produces exactly one series per namespace.

Fixes #2751

Related: prometheus-community/helm-charts#5873

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.)

Changelog entry

Fix InfoInhibitor alert failing with duplicate series error when multiple info-level alerts fire in the same namespace.

@varundeepsaini varundeepsaini requested a review from a team as a code owner December 21, 2025 15:51
@varundeepsaini
Copy link
Copy Markdown
Contributor Author

@simonpasquier pushed the suggestion

@varundeepsaini varundeepsaini force-pushed the fix/infoinhibitor-duplicate-series branch from 1660d5f to a5be32e Compare December 22, 2025 13:40
@sebastiangaiser
Copy link
Copy Markdown
Contributor

Had the same issue. Thanks for opening the PR 🥳

@varundeepsaini varundeepsaini force-pushed the fix/infoinhibitor-duplicate-series branch from a5be32e to d0b1fdb Compare January 7, 2026 09:45
@sebastiangaiser
Copy link
Copy Markdown
Contributor

@simonpasquier friendly ping. It would be really great to get this on the road 🚀

@varundeepsaini
Copy link
Copy Markdown
Contributor Author

@simonpasquier could we merge this ?

@sebastiangaiser
Copy link
Copy Markdown
Contributor

@simonpasquier thanks for approving the PR. Could this please be merged?

@slashpai
Copy link
Copy Markdown
Contributor

you need to rebase the branch

@varundeepsaini varundeepsaini force-pushed the fix/infoinhibitor-duplicate-series branch from d0b1fdb to f9f33b9 Compare February 26, 2026 13:17
@sebastiangaiser
Copy link
Copy Markdown
Contributor

Sorry to write here again but this alert is failing for me from time to time. Any chance to get it merged now? 😅

@simonpasquier simonpasquier merged commit 06dd187 into prometheus-operator:main Mar 10, 2026
11 checks passed
@varundeepsaini varundeepsaini deleted the fix/infoinhibitor-duplicate-series branch March 10, 2026 09:36
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.

InfoInhibitor - duplicate series

4 participants