Skip to content

Clarify control plane node description in cluster introduction#56522

Open
Debasish-87 wants to merge 1 commit into
kubernetes:mainfrom
Debasish-87:fix-56452-topology-spread-docs
Open

Clarify control plane node description in cluster introduction#56522
Debasish-87 wants to merge 1 commit into
kubernetes:mainfrom
Debasish-87:fix-56452-topology-spread-docs

Conversation

@Debasish-87

Copy link
Copy Markdown

Fixes #56507

What this PR does

  • Clarifies that the production recommendation refers to control plane nodes.
  • Makes it explicit that control plane nodes are also nodes in the Kubernetes cluster.
  • Improves the wording to reduce confusion in the cluster introduction tutorial.

@kubernetes-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign nate-double-u for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubernetes-prow
kubernetes-prow Bot requested a review from nate-double-u July 18, 2026 04:25
@kubernetes-prow kubernetes-prow Bot added the language/en Issues or PRs related to English language label Jul 18, 2026
@kubernetes-prow
kubernetes-prow Bot requested a review from tengqm July 18, 2026 04:25
@kubernetes-prow kubernetes-prow Bot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jul 18, 2026
@netlify

netlify Bot commented Jul 18, 2026

Copy link
Copy Markdown

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 3a1cad8
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/6a5bc4014f12910008cf3a4b
😎 Deploy Preview https://deploy-preview-56522--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@lmktfy lmktfy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the PR. Superficially it looks consistent, but there's a problem. People reading this may get an idea about Kubernetes and its control plane that is unhelpfully misleading.

It's a subtle detail but that perhaps makes it all the more important not to teach it wrongly.

Comment thread content/en/docs/tutorials/kubernetes-basics/create-cluster/cluster-intro.md Outdated
@Debasish-87

Copy link
Copy Markdown
Author

Thanks for the detailed explanation. I understand the concern now. I based my change on the wording suggested in the linked issue, but I agree that it shouldn't imply control plane nodes are required for all Kubernetes clusters. I'll revise the PR to keep the tutorial accurate while still addressing the original confusion.

@Debasish-87
Debasish-87 force-pushed the fix-56452-topology-spread-docs branch from cc7312a to 3a1cad8 Compare July 18, 2026 18:20
@Debasish-87

Copy link
Copy Markdown
Author

Thanks again for the feedback. I've updated the wording to avoid implying that control plane nodes are required and simplified the paragraph to describe a common supported deployment model. Please let me know if you'd like me to adjust the wording further.

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. language/en Issues or PRs related to English language size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Using Minikube to Create a Cluster: Unclear paragraph.

2 participants