-
Notifications
You must be signed in to change notification settings - Fork 77
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
Flux v2.2.0: Update CRDs and component versions #204
Flux v2.2.0: Update CRDs and component versions #204
Conversation
Signed-off-by: Rafael da Fonseca <[email protected]>
Signed-off-by: Rafael da Fonseca <[email protected]>
Signed-off-by: Rafael da Fonseca <[email protected]>
Signed-off-by: Rafael da Fonseca <[email protected]>
Signed-off-by: Rafael da Fonseca <[email protected]>
Signed-off-by: Rafael da Fonseca <[email protected]>
Signed-off-by: Rafael da Fonseca <[email protected]>
8763ddd
to
8056e13
Compare
cc @stefanprodan this should help get things ready for flux v2.2.0 :) |
Please hold this until we release Flux 2.2.0 then we can bump the app version here and the cli version. |
Yeah, i was about to ask about bumping those since the release date is expected to be today right? :) |
Yeap couple of hours from now |
@rsafonseca please bump the app version and the CLI tag, we've released v2.2.0 |
Signed-off-by: Rafael da Fonseca <[email protected]>
all set @stefanprodan 👍 |
Please bump the other charts as well |
We also need to bump Kubernetes in CI to v1.28.0 here
|
Signed-off-by: Rafael da Fonseca <[email protected]>
Done, it was actually in 2 places :) |
argh, the liniting job is tying all the charts together so it's failing for the charts i didn't update :( |
Please bump the chart version for flux2-notification and flux2-sync |
There is an update, the app version so chart version needs a bump too |
… flux2-sync charts Signed-off-by: Rafael da Fonseca <[email protected]>
Yeah, just noticed that as I typed it... the Makefile target did that for me hehe.. i added the notification.toolkit.fluxcd.io apiversion bump to flux2-notification chart as a bonus :) |
Signed-off-by: Rafael da Fonseca <[email protected]>
It should be all green now :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @rsafonseca 🏅
What this PR does / why we need it:
Updates the CRDs to include the latest specs, updates components to latest versions
Special notes for your reviewer:
Allows the helm-chart to be used for proper install/function of the latest component versions
Checklist
make reviewable