Updated Set up kubectl on Linux docs and bug fix #43481
Conversation
k8s-ci-robot
added
the
cncf-cla: yes
k8s-ci-robot
added
language/en
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
|
||
| 1. Install kubectl using `zypper`: | ||
|
|
||
| 2. Install kubectl using `zypper`: | ||
| ```bash | ||
| sudo zypper install -y kubectl |
There was a problem hiding this comment.
@Community-Programmer, close it with ```.
There was a problem hiding this comment.
Ok thank you for the guidance will copy the same
There was a problem hiding this comment.
We already have PR #43444, which was raised earlier to address the layout problem in this page. The changes made here somewhat duplicate with the ones in that PR.
There was a problem hiding this comment.
Thank you for bringing this to my attention. I've reviewed PR #43444, which was raised earlier to address layout problems on this page, and I understand that the changes in my current PR may somewhat duplicate those in PR #43444.
To ensure the most efficient and coordinated approach, could you please advise whether it would be best to remove the duplicated changes from my current PR? If there are any unique aspects in my changes that could still be beneficial, I'd be happy to discuss how to incorporate them effectively.
|
@Community-Programmer Thanks for your contribution. This PR seems to affect files in different languages (English and Chinese). However, the documentation process varies for each localization, and we usually prefer not to merge PRs that affect multiple languages at once. It's advisable to focus on a single language in a single PR. I hope this suggestion is helpful. |
|
Applying hold until PR targets only single language. /hold |
k8s-ci-robot
added
the
do-not-merge/hold
Thank you for your feedback and guidance. I appreciate the suggestion to focus on a single language in a single PR to streamline the documentation process. I'll take this advice to heart and will be splitting this PR into multiple PRs, each dedicated to a specific language. To facilitate this, I'll create a new branch and proceed accordingly. I'll keep you updated once the language-specific PRs are ready for review. |
|
@dipesh-rawat @utkarsh-singh1 Should I create a new PR for the same |
k8s-ci-robot
added
the
needs-rebase
|
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Community-Programmer
force-pushed
the
main
branch
from
a6c4897 to
b9bb81d
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
k8s-ci-robot
added
size/XS
Subject: New PR for 'Install and Set Up kubectl on Linux' Documentation Fixes
Solves the issue : #43366
Dear Reviewers and Maintainers,
I closed PR #43393 and opend this one because I've made further improvements and fixes to the 'Install and Set Up kubectl on Linux' documentation. I believe that these changes will significantly enhance the quality and accuracy of the documentation.
Fix 1:
fixed the typo error in the zn-ch docs for Installation and Setting Up kubectl on Linux
Fix 2:
Translated the new tab about SUSE-based distributions (zn-ch)
Fix 3:
Fixed the content merging of two tabs in Install using native package management
Fix 4:
Removed the unwanted html tag in the bash (en)
I apologize for any confusion caused by the previous PR. I am committed to ensuring that the documentation is as helpful and reliable as possible and am grateful for your assistance in reviewing and merging these changes.
Your feedback and suggestions are highly appreciated