cilium / cilium-etcd-operator

Operator to manage Cilium's etcd cluster
Apache License 2.0
26 stars 15 forks source link

CoreDNS 1.8.4 breaks cilium-etcd causing check-dns initcontainers to fail #74

Closed eversC closed 2 years ago

eversC commented 3 years ago

Versions of CoreDNS < 1.8.4 contained a bug that caused non-ready pod IPs to be returned when doing a DNS lookup for a headless k8s service.

CoreDNS v1.8.4 contained a fix for this, which caused the initContainers of our cilium-etcd pods to start failing, as the DNS lookups were no longer succeeding.

I think etcd-operator will need changing to set the publishNotReadyAddresses field in the k8s service spec.

Seeing as https://github.com/coreos/etcd-operator is no longer actively developed or maintained, would it make sense for Cilium to fork that repo and publish their own docker image (with the aforementioned etcd-operator fix)?

eversC commented 2 years ago

closing as "cilium/etcd-operator is no longer actively developed as Cilium does not require a KVStore to be setup in order to run."

https://cilium.slack.com/archives/C1MATJ5U5/p1637754020020900