Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update HelmRelease, Alerts and Providers to Flux v2.2.0 #1718

Merged
merged 2 commits into from
Dec 12, 2023
Merged

Conversation

darkowlzz
Copy link
Contributor

@darkowlzz darkowlzz commented Nov 14, 2023

Notification-controller is introducing v1beta3 APIs for Alerts and Providers, refer fluxcd/notification-controller#540. Helm-controller is introducing v2beta2 API for HelmRelease. Update all the references to those APIs to use the new version.

Depends on fluxcd/notification-controller#540 and fluxcd/helm-controller#738.

@darkowlzz darkowlzz added the area/docs Documentation related issues and pull requests label Nov 14, 2023
@darkowlzz darkowlzz marked this pull request as ready for review November 28, 2023 11:19
@darkowlzz darkowlzz changed the title Update Alerts and Providers to v1beta3 API Update HelmRelease, Alerts and Providers to Flux v2.2.0 Dec 5, 2023
@darkowlzz
Copy link
Contributor Author

Added changes related to HelmRelease v2beta2 API as well.

@makkes makkes added the hold Issues and pull requests put on hold label Dec 6, 2023
@makkes
Copy link
Member

makkes commented Dec 6, 2023

This needs to be held until 2.2 is actually released.

Notification-controller is introducing v1beta3 APIs for Alerts and
Providers. Update all the references to those APIs to use the new
version.

Signed-off-by: Sunny <darkowlzz@protonmail.com>
Signed-off-by: Sunny <darkowlzz@protonmail.com>
@makkes makkes removed the hold Issues and pull requests put on hold label Dec 12, 2023
@makkes makkes merged commit 7420f13 into main Dec 12, 2023
5 checks passed
@makkes makkes deleted the nc-v1beta3 branch December 12, 2023 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs Documentation related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants