Skip to content

fix: correct typos and update stale kubectl --validate advice in debug-pods#55297

Open
EoinTrial wants to merge 1 commit intokubernetes:mainfrom
EoinTrial:fix/doc-improvements
Open

fix: correct typos and update stale kubectl --validate advice in debug-pods#55297
EoinTrial wants to merge 1 commit intokubernetes:mainfrom
EoinTrial:fix/doc-improvements

Conversation

@EoinTrial
Copy link
Copy Markdown
Contributor

This PR makes two types of improvement, identified and fixed via Claude and codespell:

Content update (debug-pods.md)
The "My pod is running but not doing what I told it to do" section contained stale advice about kubectl apply --validate, so I've:

  • updated --validate to --validate=strict, reflecting that kubectl now defaults to server-side field validation (GA since Kubernetes 1.27)
  • replaced the old client-side schema.go error output with the current server-side validation error format
  • removed a stale TODO comment referencing Swagger validation: stop ignoring unknown fields kubernetes#11914, which was merged years ago

Typo fixes (4 files)

  • node-autoscaling.md: homogenous → homogeneous
  • pod-certificate-request-v1beta1.md: contructed → constructed
  • custom-resource-definition-v1.md: metdata → metadata
  • device-class-v1.md: classses → classes

@k8s-ci-robot
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 lmktfy 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

@k8s-ci-robot k8s-ci-robot added the language/en Issues or PRs related to English language label Apr 8, 2026
@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Apr 8, 2026
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 8, 2026

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit bd8b22e
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/69d694aaef51c90008d4add4
😎 Deploy Preview https://deploy-preview-55297--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.

@CodesbyUnnati
Copy link
Copy Markdown

LGTM

Copy link
Copy Markdown
Member

@Arhell Arhell left a comment

Choose a reason for hiding this comment

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

Lgtm

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/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants