fluxcd-community / helm-charts

Community maintained Helm charts for Flux
Apache License 2.0
121 stars 76 forks source link

Flux v2.2.0: Update CRDs and component versions #204

Closed rsafonseca closed 10 months ago

rsafonseca commented 10 months ago

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

rsafonseca commented 10 months ago

cc @stefanprodan this should help get things ready for flux v2.2.0 :)

stefanprodan commented 10 months ago

Please hold this until we release Flux 2.2.0 then we can bump the app version here and the cli version.

rsafonseca commented 10 months ago

Yeah, i was about to ask about bumping those since the release date is expected to be today right? :)

stefanprodan commented 10 months ago

Yeap couple of hours from now

stefanprodan commented 10 months ago

@rsafonseca please bump the app version and the CLI tag, we've released v2.2.0

rsafonseca commented 10 months ago

all set @stefanprodan 👍

stefanprodan commented 10 months ago

Please bump the other charts as well

stefanprodan commented 10 months ago

We also need to bump Kubernetes in CI to v1.28.0 here https://github.com/fluxcd-community/helm-charts/blob/9d38c2932cecef00fc7d6eb76cb83f4f78b08109/.github/workflows/e2e-sync.yaml#L21

rsafonseca commented 10 months ago

Done, it was actually in 2 places :)

rsafonseca commented 10 months ago

argh, the liniting job is tying all the charts together so it's failing for the charts i didn't update :(

stefanprodan commented 10 months ago

Please bump the chart version for flux2-notification and flux2-sync

stefanprodan commented 10 months ago

argh, the liniting job is tying all the charts together so it's failing for the charts i didn't update :(

There is an update, the app version so chart version needs a bump too

rsafonseca commented 10 months ago

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

rsafonseca commented 10 months ago

It should be all green now :)