-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
kubectl: migrate $HOME/.kube to $HOME/.config/kube #97885
Conversation
@dougsland: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. 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. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: dougsland 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 |
/sig cli |
/retest |
/test pull-kubernetes-e2e-gce-100-performance |
/test pull-kubernetes-e2e-gce-ubuntu-containerd |
/retest |
1 similar comment
/retest |
/kind feature |
/retest |
1 similar comment
/retest |
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/hold
this causes compatibility issues for client-go consumers
/cc @lavalamp |
This migration seems super painful, what is the benefit? The KEP states that it's a non-goal to "Deprecate any file under $HOME/.kube/"? The KEP is also not merged yet? |
Even just adding a new place to search has the potential to be very confusing to users and e.g. change behavior should that "new" location somehow already exist on their system. |
Hi @lavalamp. Thanks for your review.
Use the XDG Spec as must of programs use these days. Users have been asking. However, I will state all possibilities in the KEP in a few minutes as Jordan requested. They can even use:
Yes, the patch won't create new files names.
No, people are reviewing. |
It should documented, also that's why we have this code in the patch reusing code:
|
This patch implement the migration config from $HOME/.kube to use XDG Base Directory Specification. The recommended path now will be $HOME/.config/kube. KEP: kubernetes/enhancements#2111 Signed-off-by: Douglas Schilling Landgraf <dougsland@redhat.com>
@dougsland: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. I understand the commands that are listed here. |
I read the comments here, and I don't follow the argument for why the disruption is worth the benefit. I would vote against doing this. Improving the existing documentation is fine. |
adding some other folks from sig-cli to the thread: |
I agree with this. |
I've previously asked for the ability to support a directory of per-cluster kubeconfigs. Perhaps we could support a directory of kubeconfigs under |
@justinsb that's a good idea. |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
The k8s community decided to not move forward with the KEP regarding this PR. |
kubectl: migrate $HOME/.kube to $HOME/.config/kube
This patch implements the migration config from $HOME/.kube to
use XDG Base Directory Specification. The recommended
path now will be $HOME/.config/kube.
KEP: kubernetes/enhancements#2111
Signed-off-by: Douglas Schilling Landgraf dougsland@redhat.com