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

Impossible to deploy on Kubernetes < v1.22 due to a hard version constraint in Chart.yaml #913

Closed
vpedosyuk opened this issue Jun 15, 2023 · 2 comments · Fixed by #916
Closed
Labels
bug Something isn't working

Comments

@vpedosyuk
Copy link

Describe the bug
The README says:

Kubernetes 1.22+ - This is the earliest version of Kubernetes tested. It is possible that this chart works with earlier versions but it is untested.

However, in fact, it's impossible to run it on, let's say, Kubernetes v1.21 because there's an explicit version check:
https://github.com/hashicorp/vault-helm/blob/main/Chart.yaml#L8

Consequently, I get the following error:

dry-run upgrade for comparison failed: chart requires kubeVersion: >= 1.22.0-0 which is incompatible with Kubernetes v1.21.2

To Reproduce
Steps to reproduce the behavior:

  1. Install an older chart on Kubernetes 1.21
  2. Try upgrading to the latest chart version

Expected behavior
It's still possible to deploy the Helm chart on older Kubernetes versions but without any guarantees as stated in README.md

@vpedosyuk vpedosyuk added the bug Something isn't working label Jun 15, 2023
@tomhjp
Copy link
Contributor

tomhjp commented Jun 22, 2023

Thanks for raising this, I agree we should keep the hard requirement looser, so in #916 I've reverted the minimum required version back to 1.20, but as you say it is without testing guarantees.

@vpedosyuk
Copy link
Author

@tomhjp, thanks, v1.20 would work perfectly fine for me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants