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

docs: Consul DNS views on Kubernetes #21802

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

docs: Consul DNS views on Kubernetes #21802

wants to merge 23 commits into from

Conversation

boruszak
Copy link
Contributor

@boruszak boruszak commented Oct 2, 2024

Description

This PR updates the Consul v1.20 documentation to support the Consul DNS proxy on Kubernetes deployments.

Links

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

* backport of commit a33e903

* backport of commit 37163dc

* backport of commit 38f0907

* backport of commit 6ab7ec2

* backport of commit 7ac4178

* backport of commit 5dfebb2

* backport of commit 316d68c

---------

Co-authored-by: Sarah Alsmiller <[email protected]>
Co-authored-by: sarahalsmiller <[email protected]>
* backport of commit 888e302

* backport of commit 17499dc

* backport of commit d933d37

---------

Co-authored-by: Dhia Ayachi <[email protected]>
Co-authored-by: sarahalsmiller <[email protected]>
….20.x (#21774)

* backport of commit 4421ce1

* Upgrade ubi image to 9.4 (#21750)

---------

Co-authored-by: Michael Zalimeni <[email protected]>
Co-authored-by: Sarah Alsmiller <[email protected]>
Co-authored-by: sarahalsmiller <[email protected]>
* backport of commit 1e97297

* backport of commit b7053f5

* backport of commit a391f2f

---------

Co-authored-by: jm96441n <[email protected]>
…to release/1.20.x (#21783)

backport of commit c3db6c9

Co-authored-by: Michael Zalimeni <[email protected]>
@boruszak boruszak added type/docs Documentation needs to be created/updated/clarified pr/do-not-merge PR cannot be merged in its current form. pr/no-changelog PR does not need a corresponding .changelog entry backport/1.20 Changes are backported to 1.20 labels Oct 2, 2024
@github-actions github-actions bot added type/ci Relating to continuous integration (CI) tooling for testing or releases theme/contributing Additions and enhancements to community contributing materials labels Oct 2, 2024
@boruszak boruszak removed type/ci Relating to continuous integration (CI) tooling for testing or releases theme/contributing Additions and enhancements to community contributing materials labels Oct 2, 2024
….20.x (#21774)

* backport of commit 4421ce1

* Upgrade ubi image to 9.4 (#21750)

---------

Co-authored-by: Michael Zalimeni <[email protected]>
Co-authored-by: Sarah Alsmiller <[email protected]>
Co-authored-by: sarahalsmiller <[email protected]>
@boruszak boruszak removed the pr/do-not-merge PR cannot be merged in its current form. label Oct 8, 2024
website/content/docs/k8s/dns/views/enable.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/enable.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/enable.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/enable.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/index.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/index.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/index.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/index.mdx Outdated Show resolved Hide resolved
website/content/docs/services/discovery/dns-cache.mdx Outdated Show resolved Hide resolved
website/content/docs/services/discovery/dns-cache.mdx Outdated Show resolved Hide resolved
Copy link
Contributor

@aimeeu aimeeu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a few nits

website/content/docs/services/discovery/dns-cache.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/index.mdx Show resolved Hide resolved
website/content/docs/k8s/dns/views/index.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/index.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/enable.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/enable.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/enable.mdx Outdated Show resolved Hide resolved
website/content/docs/k8s/dns/views/enable.mdx Outdated Show resolved Hide resolved
Co-authored-by: Aimee Ukasick <[email protected]>
Co-authored-by: Blake Covarrubias <[email protected]>
Copy link
Member

@jmurret jmurret left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!

@@ -14,7 +14,7 @@ This page describes the process to return cached results in response to DNS look
By default, Consul serves all DNS results with a `0` TTL value so that it returns the most recent information. When operating at scale, this configuration may result in additional latency because servers must respond to every DNS query. There are several strategies for distributing this burden in your datacenter:

- [Allow Stale Reads](#stale-reads). Allows other servers besides the leader to answer the query rather than forwarding it to the leader.
- [Configure DNS TTLs](#ttl-values). Configure a TTL to nodes or services that the DNS subsystem on the service container’s OS uses. Services then resolve DNS queries locally without any external requests.
- [Configure DNS TTLs](#ttl-values). Configure DNS time-to-live (TTL) values for nodes or services so that the DNS subsystem on the container’s operating system can cache responses. Services then resolve DNS queries locally without any external requests.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.20 Changes are backported to 1.20 pr/no-changelog PR does not need a corresponding .changelog entry type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants