search

coredns / helm

Helm Charts for CoreDNS
Apache License 2.0
107 stars 111 forks source link

etcd-operator no longer works #69

Open AshtonKem opened 2 years ago

AshtonKem commented 2 years ago

etcd-operator was deprecated two years ago, and does not seem to still work with newer versions of k8s. Installing gives me this error:

 helm install stable/etcd-operator --generate-name
WARNING: This chart is deprecated
Error: INSTALLATION FAILED: failed to install CRD crds/etcd-cluster-crd.yaml: resource mapping not found for name: "etcdclusters.etcd.database.coreos.com" namespace: "" from "": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
chrisohaver commented 2 years ago

This is the coredns helm repo.

AshtonKem commented 2 years ago

Yes, but the instructions for this chart specify that it requires another chart that is not just broken, but deprecated. I can’t file a bug with the etcd-operator chart, it’s deprecated and archived

Given that etcd-operator is not going to come back, I think this chart needs to be updated with new instructions/dependencies.

— Ashton Kemerling

Sent from my iPhone.

On Jun 21, 2022, at 10:41 PM, chrisohaver @.***> wrote:

 This is the coredns helm repo.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.

chrisohaver commented 2 years ago

Ah. I wasn't aware of the third bullet use case...

CoreDNS as an external dns provider for kubernetes federation. This is a sub case of 'external dns service' which uses etcd plugin for CoreDNS backend. This deployment mode as a dependency on etcd-operator chart, which needs to be pre-installed.

Is this a hard dependency on etcd-operator as implied? Or does this just mean that the mode requires etcd?

IOW, should this instead say:

"This deployment mode requires an etcd cluster to be present at install time."

chrisohaver commented 2 years ago

I think the instruction intend to convey the following:

  1. This 3rd mode requires an etcd cluster.
  2. This chart does not install an ectd cluster.
  3. An etcd cluster must therefor be installed by some other means.
hagaibarel commented 1 year ago

@chrisohaver according to https://groups.google.com/g/kubernetes-sig-multicluster/c/aZv-YuafSko it seems that kubernetes federation is deprecated and no other project is slated to replace it.

Given that, should we just remove the reference to kubernetes federation from the chart use cases?