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

Virtual IPs and Service Proxies states that kube-proxy config is a ConfigMap #38168

Closed
sftim opened this issue Nov 29, 2022 · 5 comments
Closed
Labels
kind/bug Categorizes issue or PR as related to a bug. language/en Issues or PRs related to English language needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/network Categorizes an issue or PR as relevant to SIG Network.

Comments

@sftim
Copy link
Contributor

sftim commented Nov 29, 2022

This is a Bug Report

Problem:
Virtual IPs and Service Proxies states that you use a ConfigMap to configure kube-proxy.

I don't think that's right. See #36675 (comment) for more details.

Proposed Solution:

  • Confirm that the in-tree kube-proxy implementation is configured via a local file.
  • Revise the reference page to explain that, although you configure kube-proxy with a file, a cluster operator might set up a DaemonSet to define Pods that run kube-proxy, and people often use a ConfigMap.

If you judge that it's useful to, you could also add a task page about managing the DaemonSet that runs kube-proxy.

Page to Update:
https://kubernetes.io/docs/reference/networking/virtual-ips/

Additional Information:
/language en

Prompted by #36675 (comment)

@sftim sftim added the kind/bug Categorizes issue or PR as related to a bug. label Nov 29, 2022
@k8s-ci-robot k8s-ci-robot added language/en Issues or PRs related to English language needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Nov 29, 2022
@sftim
Copy link
Contributor Author

sftim commented Nov 29, 2022

/triage accepted

@k8s-ci-robot k8s-ci-robot added triage/accepted Indicates an issue or PR is ready to be actively worked on. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Nov 29, 2022
@kundan2707
Copy link
Contributor

/sig network

@k8s-ci-robot k8s-ci-robot added the sig/network Categorizes an issue or PR as relevant to SIG Network. label Nov 30, 2022
@k8s-triage-robot
Copy link

This issue has not been updated in over 1 year, and should be re-triaged.

You can:

  • Confirm that this issue is still relevant with /triage accepted (org members only)
  • Close this issue with /close

For more details on the triage process, see https://www.kubernetes.dev/docs/guide/issue-triage/

/remove-triage accepted

@k8s-ci-robot k8s-ci-robot added needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. and removed triage/accepted Indicates an issue or PR is ready to be actively worked on. labels Jan 19, 2024
@sftim
Copy link
Contributor Author

sftim commented Jan 19, 2024

Looks fixed
/close

OK to reopen it if I was wrong.

@k8s-ci-robot
Copy link
Contributor

@sftim: Closing this issue.

In response to this:

Looks fixed
/close

OK to reopen it if I was wrong.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. language/en Issues or PRs related to English language needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/network Categorizes an issue or PR as relevant to SIG Network.
Projects
None yet
Development

No branches or pull requests

4 participants