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

Move towards using standard kubernetes provider for manifests/crds #821

Closed
viniciusdc opened this issue Sep 14, 2021 · 3 comments
Closed
Labels
type: bug 🐛 Something isn't working type: enhancement 💅🏼 New feature or request

Comments

@viniciusdc
Copy link
Contributor

Recently I've made a simple test running the QHub deployment on a freshly installed machine. Basically I've set up minikube + kubectl and spun up a minikube cluster. Followed the Local Testing section of the Testing docs and it all worked fine until "Deploy and render the infrastructure" where I got the following error when deploying:

[terraform]: │ Error: Failed to determine GroupVersionResource for manifest
[terraform]: │ 
[terraform]: │   with module.qhub.module.kubernetes-dask-gateway.kubernetes_manifest.main,
[terraform]: │   on modules/kubernetes/services/dask-gateway/crds.tf line 1, in resource "kubernetes_manifest" "main":
[terraform]: │    1: resource "kubernetes_manifest" "main" {
[terraform]: │ 
[terraform]: │ cannot select exact GV from REST mapper

I've tried to search about that problem but didn't find much information regarding terraform usage. Fortunatly, I was able to solve that problem removing the docker engine installation and changing it's version from latest to 20.10.8 as well as directly specifying the kubernetes version for the cluster:
minikube start --kubernetes-version=1.19.4 --driver=docker --cpus 4 --memory 4096

There might be some changes within the latest versions of the necessary components for the local deployment, so I would suggest a review in our docs.

@viniciusdc viniciusdc added the needs: discussion 💬 Needs discussion with the rest of the team label Sep 14, 2021
@magsol magsol changed the title Failed local deployment on Ubunut fresh install Failed local deployment on Ubuntu fresh install Sep 15, 2021
@magsol
Copy link
Contributor

magsol commented Sep 15, 2021

Tried again, changing only the kubernetes version by including the flag --kubernetes-version=v1.19.4 in the minikube start command, as my Docker engine was already at version 20.10.8 (that seems like it's the current latest stable?).

It did require that I spin down and delete my existing minikube cluster, as it was running kubernetes 1.22. As such, I had to back up and repeat some of the previous steps in the Local Testing instructions to initialize the minikube cluster. As mentioned, the only step that changed was including an explicit kubernetes version in the minikube startup command.

The rest of the steps proceeded smoothly and without problems exactly as indicated.

Of course, when attempting to navigate to github-actions.qhub.dev/hub/login in the browser, Chrome nuked the connection due to an overdeveloped cert management system, but that's for a different ticket. curl was able to access the page just fine. Closing.

@magsol magsol closed this as completed Sep 15, 2021
@costrouc
Copy link
Member

Going to reopen this issue seem related to hashicorp/terraform-provider-kubernetes-alpha#235. Looks like terraform now supports a kubernetes_manifest and we should now start to use the kubernetes provider for all this work. https://www.hashicorp.com/blog/beta-support-for-crds-in-the-terraform-provider-for-kubernetes

https://github.com/hashicorp/terraform-provider-kubernetes

@costrouc costrouc reopened this Sep 17, 2021
@costrouc costrouc changed the title Failed local deployment on Ubuntu fresh install Move towards using standard kubernetes provider for manifests/crds Sep 17, 2021
@costrouc costrouc added type: bug 🐛 Something isn't working type: enhancement 💅🏼 New feature or request and removed needs: discussion 💬 Needs discussion with the rest of the team labels Sep 17, 2021
@trallard trallard moved this to Needs Triage in QHub Project Mangement 🚀 Dec 17, 2021
@costrouc
Copy link
Member

Done in #1003. Closing issue.

Repository owner moved this from Needs Triage 🔍 to Done 💪🏾 in QHub Project Mangement 🚀 Feb 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug 🐛 Something isn't working type: enhancement 💅🏼 New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants