Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve DNS documentation #6441

Closed
wants to merge 95 commits into from
Closed

Improve DNS documentation #6441

wants to merge 95 commits into from

Conversation

tengqm
Copy link
Contributor

@tengqm tengqm commented Nov 24, 2017

This PR improves the DNS documentation:

  • Move DNS debuggin and problem diagnosis content from concepts to tasks
  • Add description about all DNS policies
  • Add section about dnsConfig
  • Move inheriting DNS from node to tasks to keep the concepts page clean
  • Move outdated discussion about v1.2 and v1.3 into a 'compatibility' subsection as references only
  • Changed page title to better summarize the contents
  • Polish command line examples by merging input and outputs when appropriate.

This change is Reviewable

zacharysarah and others added 30 commits October 19, 2017 16:12
This is kind of jarring / missing an article. I'm guessing it should either be ' to a rack of bare metal servers.' or '...to racks of bare metal servers.'.
 - support request
 - code bug report
Fix the not-working test case yaml for /doc/concepts/storage/volumes.md
Some edits for clarity and condensed language.
Fix format.
The url no need. just  highlight.
Fix leading spaces in commands.
I believe this should be "users" not "uses"
guarnatees  to guarantees
chenopis and others added 10 commits November 9, 2017 15:38
…elease-1.9

* 'master' of https://github.com/kubernetes/website:
  Rename `gems` to `plugins` in `/_config.yml`
  fix test dependency
  Fix a SVG which received many complaints (kubernetes#6252)
  update docs to reference latest updates
  Update daemonset.md
  [Monthly Maintenance] Fix broken links from Screaming Frog SEO Spider
  add note that encryption at rest is supported
  call out missing content in effort to fix it
…elease-1.9

* 'master' of https://github.com/kubernetes/website:
  Update PodAffinity/PodAntiAffinity Use-Case (kubernetes#6291)
  translate doc distribute-credentials-secure into Chinese (kubernetes#6292)
  Clarify GPU resource requests and limits (kubernetes#6294)
  Rename Google Container Engine to Google Kubernetes Engine
  Update debug-services doc
  New Partners Section for Conformance (kubernetes#6182)
  Modify the translation for better expression as suggestion.
  remove unnecessary "or" in series
  Modify the translation for better expression.
  Document unconfined apparmor profile (kubernetes#6269)
  Fix taint names used in admission controllers (kubernetes#6264)
  Document nics req
  fix a typo
  The correct name is [Route] and not [Router]
  Update service.md
  use space instead of newline for dnf compatibility
  Add the translated document 'The Kubernetes API' - Chinese version.
Documented service annotation for AWS ELB SSL policy
Weekly maintenance 11/22/17
This is a new flag for the `kubeadm token create` command.
@k8s-ci-robot k8s-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Nov 24, 2017
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Nov 24, 2017

Deploy preview ready!

Built with commit 76d63e2

https://deploy-preview-6441--kubernetes-io-master-staging.netlify.com

@MrHohn
Copy link
Member

MrHohn commented Nov 28, 2017

Will review this soon.
/assign

@tengqm
Copy link
Contributor Author

tengqm commented Nov 29, 2017

Am I supposed to close this one now? A copy of the PR (#6479) is now proposed to release-1.9 branch.

@zacharysarah
Copy link
Contributor

@tengqm 👋 Yes, this PR should close. (I'll close it now.) Thanks for opening #6479.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.