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

101_initial_cluster not compatible Kubernetes 1.16 #5411

Closed
fly2F opened this issue Nov 6, 2019 · 11 comments
Closed

101_initial_cluster not compatible Kubernetes 1.16 #5411

fly2F opened this issue Nov 6, 2019 · 11 comments
Milestone

Comments

@fly2F
Copy link

fly2F commented Nov 6, 2019

helm install ../../helm/vitess -f 101_initial_cluster.yaml

Error: validation failed: [unable to recognize "": no matches for kind "Deployment" in version "apps/v1beta1", unable to recognize "": no matches for kind "StatefulSet" in version "apps/v1beta1", unable to recognize "": no matches for kind "EtcdCluster" in version "etcd.database.coreos.com/v1beta2"]

@fly2F
Copy link
Author

fly2F commented Nov 6, 2019

Continued deprecation of extensions/v1beta1, apps/v1beta1, and apps/v1beta2 APIs; these extensions will be retired in 1.16!

All resources under apps/v1beta1 and apps/v1beta2 - use apps/v1 instead

@morgo morgo added the Type: Bug label Nov 7, 2019
@parthibd
Copy link

Faced the same bug today while trying out the examples.

@morgo morgo self-assigned this Nov 28, 2019
@morgo
Copy link
Contributor

morgo commented Nov 28, 2019

I took a look at this today. There will be a bit of work to do, because the etcd-operator we suggest using also uses the deprecated APIs. It looks like they have a pull request waiting to merge, which worked for me: coreos/etcd-operator#2127

@SanmuWangZJU
Copy link

The latest update of etcd-operator was ten months ago, it
seems no one maintains this project.
What can we do to adapt helm3 and k8s 1.16+ ?

@morgo
Copy link
Contributor

morgo commented Mar 5, 2020

We spoke about this on the Vitess monthly call. The logical fix to me, is to switch the default topo to "kubernetes" (see #5703 ). We can then run without the etcd-operator.

The k8s topo is in review now. From there it is hopefully straight forward to fix..

@morgo morgo removed their assignment Mar 5, 2020
@ErikTMA
Copy link

ErikTMA commented Mar 20, 2020

Do you have any ETA on this? I am looking into implementing Vitess for the first time, but I am unsure if I should wait for the fix for Helm3+Kubernetes 1.16 or implement it in an older cluster.

@morgo
Copy link
Contributor

morgo commented Mar 20, 2020

We just merged the k8s topo 3 days ago. We are looking for testers, but @carsonoid mentioned on the Vitess monthly call yesterday he would like to make one more change (compress values before storing in k8s).

Personally, I would recommend trying an older cluster. If you are deploying in production on AWS/GCP/Azure etc. they are still using the older versions too.

@ErikTMA
Copy link

ErikTMA commented Mar 20, 2020

That was a fast response :o

I am using digitalocean currently, but not sure if I am going to stick with it in the long haul.

Thanks, I will setup a 1.15 cluster (assuming it works on that)

@carsonoid
Copy link
Contributor

Do you have any ETA on this? I am looking into implementing Vitess for the first time, but I am unsure if I should wait for the fix for Helm3+Kubernetes 1.16 or implement it in an older cluster.

@ErikTMA like @morgo said: I have one small but breaking change I want to make to the k8s topo implementation. But once that's in we plan on moving all the helm examples to use k8s as the default topo which should allow you to use newer clusters and get rid of our dependency on the etcd-operator.

@yiliaofan
Copy link

Do you have any ETA on this? I am looking into implementing Vitess for the first time, but I am unsure if I should wait for the fix for Helm3+Kubernetes 1.16 or implement it in an older cluster.

@ErikTMA like @morgo said: I have one small but breaking change I want to make to the k8s topo implementation. But once that's in we plan on moving all the helm examples to use k8s as the default topo which should allow you to use newer clusters and get rid of our dependency on the etcd-operator.

I encountered the same problem above. Thank you for your great work ~
You try to put aside etcd-operator, can you test ok with the latest version of k8s? thank you very much!

@sougou
Copy link
Contributor

sougou commented Apr 11, 2020

I just tested against 1.18, and it works fine. And #6052 is out showing the new way to reshard.

@morgo morgo closed this as completed Apr 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants