admin: dns: merge from the build dir#1251
Merged
devin-donnelly merged 1 commit intokubernetes:masterfrom Oct 5, 2016
Merged
Conversation
|
cc @thockin who originally wrote the build/kube-dns README |
thockin
approved these changes
Sep 19, 2016
devin-donnelly
suggested changes
Sep 20, 2016
|
|
||
| ## What things get DNS names? | ||
|
|
||
| Every Service defined in the cluster (including the DNS server itself) will be |
Contributor
There was a problem hiding this comment.
Try to avoid future tense "will be".
"Every Service defined in the cluster (including the DNS server itself) is assigned a DNS name."
| When enabled, pods are assigned a DNS A record in the form of `pod-ip-address.my-namespace.pod.cluster.local`. | ||
|
|
||
| For example, a pod with ip `1.2.3.4` in the namespace `default` with a dns name of `cluster.local` would have an entry: `1-2-3-4.default.pod.cluster.local`. | ||
|
|
| Given a Pod with the hostname set to "foo" and the subdomain set to "bar", and a headless Service named "bar" in the same namespace, the pod will see it's own FQDN as "foo.bar.my-namespace.svc.cluster.local". DNS will serve an A record at that name, pointing to the Pod's IP. | ||
|
|
||
| With v1.2, the Endpoints object also has a new annotation `endpoints.beta.kubernetes.io/hostnames-map`. Its value is the json representation of map[string(IP)][endpoints.HostRecord], for example: '{"10.245.1.6":{HostName: "my-webserver"}}'. | ||
| If the Endpoints are for a headless service, then A records will be created with the format <hostname>.<service name>.<pod namespace>.svc.<cluster domain> |
| name: busybox | ||
| ``` | ||
|
|
||
| If there exists a headless service in the same namespace as the pod and with the same name as the subdomain, the cluster's KubeDNS Server will also return an A record for the Pod's fully qualified hostname. |
Contributor
There was a problem hiding this comment.
will also return -> also returns
| ``` | ||
|
|
||
| If there exists a headless service in the same namespace as the pod and with the same name as the subdomain, the cluster's KubeDNS Server will also return an A record for the Pod's fully qualified hostname. | ||
| Given a Pod with the hostname set to "foo" and the subdomain set to "bar", and a headless Service named "bar" in the same namespace, the pod will see it's own FQDN as "foo.bar.my-namespace.svc.cluster.local". DNS will serve an A record at that name, pointing to the Pod's IP. |
Contributor
There was a problem hiding this comment.
will serve -> serves
| `pod.beta.kubernetes.io/hostname` annotation value. | ||
|
|
||
| v1.2 introduces a beta feature where the user can specify a Pod annotation, `pod.beta.kubernetes.io/subdomain`, to specify what the Pod's subdomain should be. | ||
| If the annotation is specified, the fully qualified Pod hostname will be "<hostname>.<subdomain>.<pod namespace>.svc.<cluster domain>". |
Contributor
There was a problem hiding this comment.
Passive voice.
If the annotation is specified -> If you specify the annotation
| Currently when a pod is created, its hostname is the Pod's `metadata.name` value. | ||
|
|
||
| With v1.2, users can specify a Pod annotation, `pod.beta.kubernetes.io/hostname`, to specify what the Pod's hostname should be. | ||
| If the annotation is specified, the annotation value takes precendence over the Pod's name, to be the hostname of the pod. |
Contributor
There was a problem hiding this comment.
Passive voice.
If the annotation is specified -> If you specify the annotation
"the annotation value takes precendence over the Pod name and becomes the hostname for that Pod."
a1d586d to
885fb63
Compare
Contributor
Author
|
@devin-donnelly thanks, I think I addresses all of your feedback. |
devin-donnelly
suggested changes
Sep 21, 2016
Contributor
devin-donnelly
left a comment
There was a problem hiding this comment.
Just one more nit; I think you got stuck between sentence edits here.
| configured to tell individual containers to use the DNS Service's IP to resolve DNS names. | ||
|
|
||
| Every Service defined in the cluster (including the DNS server itself) will be | ||
| A scheduled DNS Pod and Service on the cluster, and the kubelets configured to |
Contributor
There was a problem hiding this comment.
Looks like this is an incomplete sentence. How about:
"Kubernetes DNS schedules a DNS Pod and Service on th cluster, and configures the kubelets to tell individual containers to use the DNS Service's IP to resolve DNS names."
devin-donnelly
approved these changes
Sep 21, 2016
Contributor
|
Just one more nit, @philips . |
885fb63 to
dfee30a
Compare
Try and clean this mess up where the build dir has user facing docs and move them here. Ref: kubernetes/kubernetes#32931
dfee30a to
8aa10a6
Compare
Contributor
Author
|
@devin-donnelly Addressed feedback, thanks for the review! |
Contributor
Author
|
@devin-donnelly What is the next step here? Sorry to keep bugging but I want to get this cleaned up. |
mikutas
pushed a commit
to mikutas/k8s-website
that referenced
this pull request
Sep 22, 2022
Signed-off-by: Charles Pretzer <charles@buoyant.io>
Okabe-Junya
pushed a commit
to Okabe-Junya/website
that referenced
this pull request
Dec 4, 2023
Signed-off-by: Mr. Erlison <sisal.cordame_0a@icloud.com> Signed-off-by: Mr. Erlison <sisal.cordame_0a@icloud.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Try and clean this mess up where the build dir has user facing docs and move them here.
Ref: kubernetes/kubernetes#32931
This change is