Skip to content

Feat: Support api_type for Jira receiver in Alertmanager config secret#8218

Merged
simonpasquier merged 7 commits intoprometheus-operator:mainfrom
nutmos:feat/add-jira-apitype-in-am-config-secret
Jan 6, 2026
Merged

Feat: Support api_type for Jira receiver in Alertmanager config secret#8218
simonpasquier merged 7 commits intoprometheus-operator:mainfrom
nutmos:feat/add-jira-apitype-in-am-config-secret

Conversation

@nutmos
Copy link
Contributor

@nutmos nutmos commented Dec 28, 2025

Description

This PR is separated for adding only api_type in Jira receiver under Alertmanager config secret for easier to review.

Relate #7971

Type of change

What type of changes does your code introduce to the Prometheus operator? Put an x in the box that apply.

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

Verification

Unit Testing

Changelog entry

- Add api_type field in Jira receiver under Alertmanager config secret

@nutmos nutmos requested a review from a team as a code owner December 28, 2025 07:49
@nutmos nutmos force-pushed the feat/add-jira-apitype-in-am-config-secret branch from 7411542 to ba96cf2 Compare December 28, 2025 08:00
@pull-request-size pull-request-size bot added size/S and removed size/L labels Dec 28, 2025
@pull-request-size pull-request-size bot added size/M and removed size/S labels Dec 28, 2025
@nutmos nutmos changed the title Feat: Add api_type for Jira receiver in Alertmanager config secret Feat: Support api_type for Jira receiver in Alertmanager config secret Dec 28, 2025
to fix mandatory field issue
@nutmos
Copy link
Contributor Author

nutmos commented Dec 28, 2025

Hi @simonpasquier and @heliapb, I separated this from #7971 for merging only the new field for Alertmanager config secret first as it should be easier for review.

Please have a look and suggest.

Copy link
Contributor

@simonpasquier simonpasquier left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Shouldn't we also check the value of the field which should be one of auto, cloud or datacenter?

to prevent unexpected value in api_type
@nutmos
Copy link
Contributor Author

nutmos commented Jan 5, 2026

@simonpasquier Updated

to fix the validation issue
@nutmos nutmos requested a review from simonpasquier January 6, 2026 00:51
to cover wrong api type detection
Copy link
Contributor

@simonpasquier simonpasquier left a comment

Choose a reason for hiding this comment

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

Thanks!

@simonpasquier simonpasquier merged commit eb46062 into prometheus-operator:main Jan 6, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants