kubeversion compare issue

[emouawad]

Name and Version

vaultwarden 0.2.1

What architecture are you using?

None

What steps will reproduce the bug?

Clean new installation

Are you using any custom parameters or values?

helm install --namespace vaultwarden --create-namespace `
--repo https://fmjstudios.github.io/helm/ `
--values values.yml `
vaultwarden vaultwarden --debug

What is the expected behavior?

Installation successful

What do you see instead?

Error: INSTALLATION FAILED: chart requires kubeVersion: >=1.26 which is incompatible with Kubernetes v1.29.6-gke.1254000 helm.go:84: [debug] chart requires kubeVersion: >=1.26 which is incompatible with Kubernetes v1.29.6-gke.1254000

Additional information

kubeversion v1.29 is more recent than 1.26 - should be accepted to install

[mvprowess]

Hey @emouawad , first of all thanks for opening this issue to improve my charts.

This is very weird. Id like to ask you to do two things:

• Try a regular installation with helm repo add fmjstudios https://fmjstudios.github.io/helm like detailed in the projects README. And provide me with the output of that.
• Could you send me a chart or two, which you are able to install using the method you showed? I'd like to check what they specify as their kubeVersion.

[emouawad]

Nothing special about the way i am installing it. I downloaded ur chart and installed it from folder getting same error. Only when i removed the kubeVersion attribute it succeeded.

It's definitely the format of that value - i suspect it should be another format. Currently i am not on my dev machine but using the same command i installed 60+ chart such as prometheus-stack, bitnami charts etc

[mvprowess]

Hey @emouawad, thanks for the information! I double checked with the charts you mentioned and they indeed use a different format like >=1.21.0-0 to define the kubeVersion. This seems to enable the snapshot-like versioning used for GKE clusters in addition to the normal SemVer definition.

I will issue a patch version update for all charts in the coming hours where I update the kubeVersion to the format mentioned above. Would you mind testing if the installation works then?

[emouawad]

Sure! thanks for the fast response ❤️

[mvprowess]

Hey @emouawad, I just published those patch versions for all charts alongside our new keycloak-operator chart. I'll be waiting for your feedback, if that fixed the issue 🙌🏼

[emouawad]

Yes, that fixes it - thanks

[mvprowess]

Thank you for your input! 😄 I was even looking up how to integrate GKE, AKS, EKS compatibility into my CI not too long ago. Needless to say your input was crucial in improving my charts' compatibility 🙌🏼