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

Upgrade kubeVersion requirement on Livy helm charts #51

Closed
rakeshramakrishnan opened this issue Dec 7, 2020 · 7 comments
Closed

Upgrade kubeVersion requirement on Livy helm charts #51

rakeshramakrishnan opened this issue Dec 7, 2020 · 7 comments

Comments

@rakeshramakrishnan
Copy link
Contributor

rakeshramakrishnan commented Dec 7, 2020

Tried installing the latest helm charts (commit hash)

Got the following error:

❯ helm upgrade --install livy --namespace livy jahstreet/livy                                                                                                                   
Release "livy" does not exist. Installing it now.
Error: chart requires kubeVersion: 1.11.0 - 1.18.0 which is incompatible with Kubernetes v1.18.8

Can see that kubeVersion requirements have been upgraded for spark-cluster. Can we upgrade the same for Livy too?

@jahstreet
Copy link
Collaborator

Right. Thank you for catching. In the meantime you can patch it locally and create a fix PR. I can get to it in the coming days/weeks and push the fix otherwise.

@rakeshramakrishnan
Copy link
Contributor Author

Added a PR here

@andreas-eberle
Copy link

@jahstreet: Is there a reason to limit the kubernetes version to 1.18.x? My docker desktop has a kubernetes version 1.19 and I'm wondering if or what is the reason behind this limit. Furthermore, we plan to upgrade our cluster to 1.19 and thus it will become an issue there as well. I haven't found such a limitation for spark itself yet.

Thanks for your help!

@jahstreet
Copy link
Collaborator

Hi @andreas-eberle , indeed there is a reason. Please check this comment for the details.
To play with the charts locally I would suggest you to install Minikube using which you can control the API version of the local Kubernetes cluster. Please refer the guide for additional details.
Does that option solve you issue?

@Minutis
Copy link

Minutis commented Jan 6, 2021

I think this is fixed in #52 (2.0.1 chart version).

@rangnar14
Copy link

@jahstreet Currently the latest changes in your repo for customized livy along with the helm charts were working for me with k8 version 1.21 on EKS.
But now we upgraded to 1.22 which broke the setup since, the API version for ingress has changed. Can you please provide guidance here on the patch I could do?
In parallel I am also exploring on the required changes.

@jahstreet
Copy link
Collaborator

Closing this issue in favour of #82 within which it will be resolved, lets continue there.

And thanks for the patience 🙏 .

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

No branches or pull requests

5 participants