Skip to content

Conversation

@siavashs
Copy link
Contributor

@siavashs siavashs commented Oct 23, 2025

  • add Name field to config.InhibitRule
  • add Name field to inhibit.InhibitRule
  • update docs for inhibit_rule

The name will be used in new metrics to be added in a separate change.

@siavashs siavashs force-pushed the feat/add-inhibit-rule-name branch from f5c83fa to 55983d9 Compare October 23, 2025 15:56
@siavashs siavashs marked this pull request as ready for review October 23, 2025 15:59
@siavashs siavashs force-pushed the feat/add-inhibit-rule-name branch from 55983d9 to cf52d50 Compare October 23, 2025 17:08
- add Name field to config.InhibitRule
- add Name field to inhibit.InhibitRule
- update docs for inhibit_rule

The name will be used in new metrics to be added in a separate change.

Signed-off-by: Siavash Safi <siavash@cloudflare.com>
@siavashs siavashs force-pushed the feat/add-inhibit-rule-name branch from cf52d50 to 615d0f1 Compare October 23, 2025 17:10
Copy link

@MichaHoffmann MichaHoffmann left a comment

Choose a reason for hiding this comment

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

LGTM! Empty name by default makes sense to me too. Especially if we want to use this for metrics eventually.

@sysadmind sysadmind merged commit 1f2df03 into prometheus:main Oct 23, 2025
7 checks passed
@siavashs siavashs deleted the feat/add-inhibit-rule-name branch October 23, 2025 18:07
holger-waschke pushed a commit to holger-waschke/alertmanager that referenced this pull request Nov 1, 2025
- add Name field to config.InhibitRule
- add Name field to inhibit.InhibitRule
- update docs for inhibit_rule

The name will be used in new metrics to be added in a separate change.

Signed-off-by: Siavash Safi <siavash@cloudflare.com>
Signed-off-by: Holger Waschke <holger.waschke@dvag.com>
@SoloJacobs SoloJacobs mentioned this pull request Nov 24, 2025
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.

4 participants