Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

Docs need updating for CoreDNS #68

Closed
nfisher opened this issue Nov 22, 2018 · 4 comments
Closed

Docs need updating for CoreDNS #68

nfisher opened this issue Nov 22, 2018 · 4 comments
Labels
documentation Request for further documentation

Comments

@nfisher
Copy link

nfisher commented Nov 22, 2018

Will try to make a PR but currently the docs relating to consul-helm here are related to kube-dns:

https://www.consul.io/docs/platform/k8s/dns.html

I believe as of k8s v1.11 CoreDNS is now the standard name server.

https://kubernetes.io/docs/tasks/administer-cluster/dns-custom-nameservers/#coredns

Unfortunately it's a completely different syntax for coredns vs kube-dns. :(

@madsonic
Copy link

madsonic commented Nov 22, 2018

i think this part in the docs is related as well. Perhaps stating the kubernetes version that needs to upgrade to CoreDNS would be helpful.

@nfisher
Copy link
Author

nfisher commented Nov 22, 2018

Looks like GKE supports v1.11 but has kept kube-dns. Not sure what the other 1st tier cloud providers are doing?

@adilyse adilyse added the documentation Request for further documentation label Dec 4, 2018
@DWSR
Copy link

DWSR commented Feb 22, 2019

@nfisher CoreDNS is now GA and kube-dns has been deprecated per kubernetes/enhancements#566 so it makes sense to update the documentation to at least add instructions for CoreDNS

@adilyse
Copy link
Contributor

adilyse commented Mar 21, 2019

Adding CoreDNS instructions to the documentation in this Consul PR. Hopefully that helps clarify the situation for folks running CoreDNS now instead of kube-dns.

I'm going to close this, but please feel free to comment or open another issue if there is anything missing.

@adilyse adilyse closed this as completed Mar 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Request for further documentation
Projects
None yet
Development

No branches or pull requests

4 participants