[Hybrid Nodes] add details to Service Traffic Distribution guidance #994
Conversation
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
csplinter
changed the title
There was a problem hiding this comment.
Pull Request Overview
This PR updates documentation to add additional context on using Service Traffic Distribution for mixed mode clusters with both cloud and hybrid nodes.
- Updates vocabulary files by adding terms (CloudWatch, CoreDNS) for consistency.
- Revises several user guide sections to clarify recommendations and configuration steps for webhooks, CoreDNS, and CNI settings.
- Updates instructions and code snippets to reflect changes in the guidance on traffic distribution and operator deployment across hybrid and cloud nodes.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| vale/styles/config/vocabularies/EksDocsVocab/accept.txt | Adds accepted terms for CloudWatch and CoreDNS. |
| latest/ug/nodes/hybrid-nodes-webhooks.adoc | Revises language and configuration guidance for webhooks and mixed mode clusters. |
| latest/ug/nodes/hybrid-nodes-nodeadm.adoc | Adds a new section for kubelet customization with nodeadm. |
| latest/ug/nodes/hybrid-nodes-cni.adoc | Introduces new configuration for enabling service topology in the load balancer section. |
| @@ -96,35 +100,65 @@ spec: | |||
| ... | |||
| ---- | |||
| + | |||
| . Add `trafficDistribution` to the kube-dns Service configuration. | |||
| . Add the setting `trafficDistribution: PreferClose` to the `kube-dns` Service configuration to enable Topology Aware Routing. | |||
There was a problem hiding this comment.
There is an inconsistency between this instruction and the rest of the PR guidance. The title and earlier instructions suggest that Service Traffic Distribution is recommended over Topology Aware Routing. Please clarify whether the setting trafficDistribution: PreferClose is intended to enable Service Traffic Distribution or Topology Aware Routing and update the text accordingly.
Copilot uses AI. Check for mistakes.
Description of changes:
This PR adds additional context for the recommendation to use Service Traffic Distribution over Topology Aware Routing for mixed mode clusters with both cloud and hybrid nodes
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.