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

Fix Helm chart kubeVersion comparison error #34

Merged
merged 1 commit into from
Mar 18, 2022

Conversation

irasnyd
Copy link
Contributor

@irasnyd irasnyd commented Mar 15, 2022

Fixes this error when trying to install the chart:

$ helm upgrade --install vals-operator ./vals-operator-0.5.0.tgz
Release "vals-operator" does not exist. Installing it now.
Error: chart requires kubeVersion: >= 1.19 which is incompatible with Kubernetes v1.21.5-eks-bc4871b

The issue and solution is described here:
helm/helm#3810

In addition, the duplicate kubeVersion field was removed.

Fixes this error when trying to install the chart:

> $ helm upgrade --install vals-operator ./vals-operator-0.5.0.tgz
> Release "vals-operator" does not exist. Installing it now.
> Error: chart requires kubeVersion: >= 1.19 which is incompatible with Kubernetes v1.21.5-eks-bc4871b

The issue and solution is described here:
helm/helm#3810

In addition, the duplicate `kubeVersion` field was removed.
@irasnyd
Copy link
Contributor Author

irasnyd commented Mar 17, 2022

@digiserg, sorry to pester you, but you seem to be the most active contributor to this project. Could I get a review on this PR? It fixes the vals-operator Helm chart on AWS EKS.

Copy link
Collaborator

@digiserg digiserg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, it looks good.

@digiserg digiserg merged commit a2835dc into digitalis-io:master Mar 18, 2022
@digiserg
Copy link
Collaborator

@irasnyd 0.5.1 chart should be now available.

@irasnyd
Copy link
Contributor Author

irasnyd commented Mar 18, 2022

Thank you for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants