-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
add annotation to kubeconfig to identify contexts/clusters created by minikube #10126
Conversation
do we want to use hyphens instead of underscores ( |
I get your dislike of using |
I dont think it is a good idea to call it cluster-type when minikube creates the cluster, minikube clusters are not a "type" they are same clusters created by standard "kubedam" it would be misleading or confusing for normal users to see in their kubeconfig it call it "cluster_type"= minikube since minikube is gateway to most starting with kuberentes, I dont want to suggest the idea we have a special type of clusters. There is though an opportunity to create a community-wide standard naming, there doesnt seem to be any other tool using the "Extension" part of kubeconfig. |
"provider" sounds like a good suggestion |
"provider" sounds like a good suggestion |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: medyagh, priyawadhwa 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 |
good idea |
/ok-to-test |
53c73e5
to
731daf1
Compare
After this PR the following extension blocks to the context and cluster section of ~/.kube/config
closes #9970