helm icon indicating copy to clipboard operation
helm copied to clipboard

etcd-operator no longer works

Open AshtonKem opened this issue 3 years ago • 5 comments

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

AshtonKem avatar Jun 22 '22 04:06 AshtonKem

This is the coredns helm repo.

chrisohaver avatar Jun 22 '22 04:06 chrisohaver

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.

AshtonKem avatar Jun 22 '22 13:06 AshtonKem

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 avatar Jun 22 '22 14:06 chrisohaver

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.

chrisohaver avatar Jun 22 '22 14:06 chrisohaver

@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?

hagaibarel avatar Mar 28 '23 10:03 hagaibarel