-
Notifications
You must be signed in to change notification settings - Fork 14.8k
Fix link for TLS bootstrapping #34722
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
Conversation
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
I believe #34381 is already fixing one of the pages, but it is not reviewed yet. |
Oh, sorry for missed that. Hold until #34381 merged and rebase this one. |
34722bb
to
34722ab
Compare
The PR #34381 is finally merged. The reformat is already done, this PR now have only one commit. |
/lgtm |
LGTM label has been added. Git tree hash: 4a02e03890d99e44f241b4de155a664a13ae9a34
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: tengqm The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The first commit do the minimal reformat for(It's done by #34381)implementation-details.md
file. (Only adjust spaces)The second commit replaced two invalid links for TLS Bootstrapping. (ref: PR #33130)
Checked visually the same (with intended space changes).
Preview: https://deploy-preview-34722--kubernetes-io-main-staging.netlify.app/docs/reference/setup-tools/kubeadm/implementation-details/
Production: https://kubernetes.io/docs/reference/setup-tools/kubeadm/implementation-details/
/kind cleanup
/label tide/merge-method-squash