Skip to content
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

Document using SOCKS5 to access Kubernetes API #31527

Merged
merged 13 commits into from
May 2, 2022

Conversation

xens
Copy link
Contributor

@xens xens commented Jan 26, 2022

Goal of this commit is to document the usage of SOCKS5 proxies to reach the Kubernetes API.

#31497

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jan 26, 2022
@k8s-ci-robot
Copy link
Contributor

Welcome @xens!

It looks like this is your first PR to kubernetes/website 🎉. Please refer to our pull request process documentation to help your PR have a smooth ride to approval.

You will be prompted by a bot to use commands during the review process. Do not be afraid to follow the prompts! It is okay to experiment. Here is the bot commands documentation.

You can also check if kubernetes/website has its own contribution guidelines.

You may want to refer to our testing guide if you run into trouble with your tests not passing.

If you are having difficulty getting your pull request seen, please follow the recommended escalation practices. Also, for tips and tricks in the contribution process you may want to read the Kubernetes contributor cheat sheet. We want to make sure your contribution gets all the attention it needs!

Thank you, and welcome to Kubernetes. 😃

@k8s-ci-robot k8s-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 26, 2022
@k8s-ci-robot k8s-ci-robot added language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Jan 26, 2022
@netlify
Copy link

netlify bot commented Jan 26, 2022

✔️ Deploy Preview for kubernetes-io-main-staging ready!

🔨 Explore the source changes: 6323e81

🔍 Inspect the deploy log: https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/61f9911c1f1da900071f6b78

😎 Browse the preview: https://deploy-preview-31527--kubernetes-io-main-staging.netlify.app

Copy link
Member

@jihoon-seo jihoon-seo left a comment

Choose a reason for hiding this comment

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

@xens Welcome and thank you for your first PR on this repo! 😊

there's a way to have a preview of the compiled file ? I've added a Mermaid diagram and I'm not sure if it'll render properly.

Preview: https://deploy-preview-31527--kubernetes-io-main-staging.netlify.app/docs/tasks/extend-kubernetes/socks5-proxy-access-api/

I added some comments.
PTAL! 😊

@xens xens force-pushed the feat/add_socks5_documentation branch 3 times, most recently from 98bf32b to 57f6c91 Compare January 27, 2022 10:21
Copy link
Member

@jihoon-seo jihoon-seo left a comment

Choose a reason for hiding this comment

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

@xens PTAL one more time! 😊

@xens xens force-pushed the feat/add_socks5_documentation branch from 57f6c91 to d914c61 Compare January 27, 2022 12:12
@xens xens requested a review from jihoon-seo January 27, 2022 12:14
@xens
Copy link
Contributor Author

xens commented Jan 27, 2022

@jihoon-seo done :) let me know if there's anything else to fix.

@jihoon-seo
Copy link
Member

@xens Thanks!

LGTM for doc style aspect.

And it will be great if someone does a tech review on this PR! 😊

@mehabhalodiya
Copy link
Contributor

Hey @xens
I think the pull request to add these docs should target the dev-1.24 branch as per the Tim's comment.

@mehabhalodiya
Copy link
Contributor

@kubernetes/sig-cli-misc does this look OK in terms of technical correctness?

@k8s-ci-robot k8s-ci-robot added the sig/cli Categorizes an issue or PR as relevant to SIG CLI. label Jan 30, 2022
Copy link
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

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

Thanks for this PR @xens

Overall feedback:

  • nice diagram!
  • avoid using we; see other task pages for examples; most of them follow this style recommendation. Instead, address the reader as “you”.

@xens xens force-pushed the feat/add_socks5_documentation branch from d914c61 to cc8b265 Compare February 1, 2022 19:43
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 1, 2022
@xens xens force-pushed the feat/add_socks5_documentation branch from cc8b265 to f01ea2b Compare February 1, 2022 19:57
Goal of this commit is to document the usage of SOCKS5 proxies to reach
the Kubernetes API.
@xens xens force-pushed the feat/add_socks5_documentation branch from f01ea2b to 6323e81 Compare February 1, 2022 19:59
@xens xens changed the base branch from main to dev-1.24 February 1, 2022 19:59
@xens
Copy link
Contributor Author

xens commented Feb 1, 2022

Hey @xens I think the pull request to add these docs should target the dev-1.24 branch as per the Tim's comment.

Thanks it's done!

@sftim
Copy link
Contributor

sftim commented Apr 20, 2022

/milestone 1.24

@k8s-ci-robot k8s-ci-robot added this to the 1.24 milestone Apr 20, 2022
@sftim
Copy link
Contributor

sftim commented Apr 20, 2022

/retitle Document using SOCKS5 to access Kubernetes API

@k8s-ci-robot k8s-ci-robot changed the title feat: add SOCKS5 proxy documentation Document using SOCKS5 to access Kubernetes API Apr 20, 2022
@nate-double-u
Copy link
Contributor

Hello, with code freeze for k/website on Monday May 2, we need to make a decision about this PR. It appears to need some updates before getting merged in. @xens are you able to take a look?

@xens xens force-pushed the feat/add_socks5_documentation branch from e6ec645 to cb8473a Compare April 29, 2022 20:42
@nate-double-u
Copy link
Contributor

Thanks for the updates @xens!

Copy link
Contributor

@divya-mohan0209 divya-mohan0209 left a comment

Choose a reason for hiding this comment

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

This lgtm from a content perspective except a grammatical nit.

@xens xens force-pushed the feat/add_socks5_documentation branch from 9d268b1 to 378b188 Compare May 1, 2022 06:48
@aojea
Copy link
Member

aojea commented May 1, 2022

the technical part LGTM

Copy link
Contributor

@divya-mohan0209 divya-mohan0209 left a comment

Choose a reason for hiding this comment

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

lgtm. just a few non-blocking nits/suggestions.


{{< include "task-tutorial-prereqs.md" >}} {{< version-check >}}

You need SSH client software (the `ssh` tool), and an SSH server running on the remote server.
Copy link
Contributor

Choose a reason for hiding this comment

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

Referring to it as a service would help clarify, I believe.

Suggested change
You need SSH client software (the `ssh` tool), and an SSH server running on the remote server.
You need SSH client software (the `ssh` tool), and an SSH service running on the remote server.

OR

Suggested change
You need SSH client software (the `ssh` tool), and an SSH server running on the remote server.
You need SSH client software (the `ssh` tool), and an SSH server running on the remote host.

{{< include "task-tutorial-prereqs.md" >}} {{< version-check >}}

You need SSH client software (the `ssh` tool), and an SSH server running on the remote server.
You must be able to log in to the SSH server on the remote server.
Copy link
Contributor

Choose a reason for hiding this comment

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

Similar to the above.

Suggested change
You must be able to log in to the SSH server on the remote server.
You must be able to log in to the SSH server on the remote host.

OR

Suggested change
You must be able to log in to the SSH server on the remote server.
You must be able to log in to the SSH service on the remote server.


Figure 1 represents what you're going to achieve in this task.

* You have a client computer from where you're going to create requests to talk to the Kubernetes API
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want to mention here that the client computer will be referred to as local in the diagram/steps ahead? I understand we explicitly speak about how localhost is not the local client computer somewhere below. But I think it'd help if we state this explicitly for step #3.

Suggested change
* You have a client computer from where you're going to create requests to talk to the Kubernetes API
* You have a client computer, referred to as local in the steps ahead, from where you're going to create requests to talk to the Kubernetes API.

@reylejano
Copy link
Member

Applying lgtm from @aojea and @divya-mohan0209
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 2, 2022
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: eb36863142c21ae7444f20b82c4b169812c251f0

@reylejano
Copy link
Member

/approve

I'll open a follow-up PR to add suggestions

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: reylejano

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 2, 2022
@k8s-ci-robot k8s-ci-robot merged commit 7f13ad0 into kubernetes:dev-1.24 May 2, 2022
@nate-double-u
Copy link
Contributor

Thanks everyone!

@reylejano
Copy link
Member

Follow-up PR #33429

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.