fluxcd / flux2-kustomize-helm-example

A GitOps workflow example for multi-env deployments with Flux, Kustomize and Helm.
https://fluxcd.io
Apache License 2.0
935 stars 992 forks source link

Azure: source.toolkit.fluxcd.io/v1beta2 broke Arc #58

Closed alicejgibbons closed 2 years ago

alicejgibbons commented 2 years ago

The HelmRepository apiVersion update from source.toolkit.fluxcd.io/v1beta1 to source.toolkit.fluxcd.io/v1beta2 for gave me the following error when applied on cluster:

"no matches for kind \"HelmRepository\" in version \"source.toolkit.fluxcd.io/v1beta2\"

Reverting both infrastructure/sources/bitnami.yaml and infrastructure/sources/podinfo.yaml fixed the issue for me.

kingdonb commented 2 years ago

The source.toolkit.fluxcd.io/v1beta2 API version is new since Flux 0.28.0

Please upgrade your CLI binary to Flux 0.28.3 (or newer if available when you read this) – the release notes include this reference:

https://github.com/fluxcd/flux2/releases/tag/v0.28.3

Note that if you are upgrading from v0.27 you need to follow the Upgrade Flux to the Source v1beta2 API guide.

hollowimage commented 2 years ago

@kingdonb for SA:

just to note, this change completely broke the azure arc gitops tutorial as azure arc deploys versions thats incompatible with this.

https://docs.microsoft.com/en-us/azure/azure-arc/kubernetes/tutorial-use-gitops-flux2

if you guys did not intend to break it, just be aware that the blast radius for this change is massive, especially with flux being in preview mode right now with arc gitops :)

probably not the best impression now that people can't run the tutorial.

kingdonb commented 2 years ago

Thanks for the heads-up @hollowimage! I will run through the tutorial and see where I get stuck today.

stefanprodan commented 2 years ago

If Arc is on an older flux2 version, then it can’t work with this repo, we have introduced v1beta2 of the source API in flux2 v0.28.0.

hollowimage commented 2 years ago

@stefanprodan Naturally; however, I was merely attempting to inform you all about the blast radius. it would be a shame if this broken arc tutorial drove people away from fluxcd because they can't even get the basic demo working.

Perhaps it's worth collaborating with microsoft to understand their upgrade cadence, and maybe maintain a dedicated demo repo for arc. Like I said, blast radius is significant, and speaking for myself we're evaluating fluxcd as part of arc in its own right to use in other projects, and if the answer is "oh well," then we may end up looking elsewhere.

Like I said, was mostly noting for situational awareness in case you had no idea that azure arc was leveraging fluxcd.

pjbgf commented 2 years ago

Just for completeness, a new repository has been created and the documentation now points to it: https://github.com/Azure/gitops-flux2-kustomize-helm-mt

stefanprodan commented 2 years ago

@pjbgf can you please switch this repo to the latest API.