-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 option in global config to define default kube-contexts #2558
Add option in global config to define default kube-contexts #2558
Conversation
Signed-off-by: Cornelius Weig <22861411+corneliusweig@users.noreply.github.com>
Signed-off-by: Cornelius Weig <22861411+corneliusweig@users.noreply.github.com>
This command associates a skaffold.yaml with the given kube-context. Signed-off-by: Cornelius Weig <22861411+corneliusweig@users.noreply.github.com>
This command removes an association of a skaffold.yaml with a kube-context. Signed-off-by: Cornelius Weig <22861411+corneliusweig@users.noreply.github.com>
Signed-off-by: Cornelius Weig <22861411+corneliusweig@users.noreply.github.com>
Signed-off-by: Cornelius Weig <22861411+corneliusweig@users.noreply.github.com>
Signed-off-by: Cornelius Weig <22861411+corneliusweig@users.noreply.github.com>
f083476
to
b22e3c5
Compare
Signed-off-by: Cornelius Weig <22861411+corneliusweig@users.noreply.github.com>
Codecov Report
|
As discussed in #2510 (comment), the global config option for the kube-context might be an overengineering of the kube-context configurability. Therefore, I'm going to close this for now to put these changes on hold. If somebody sees an urgent need to get this functionality in, I'm happy to revive this PR. But let's first see how users get along with just the CLI option and the At the very least #2325 is affected. |
See design proposal #2384
Depends on
#2447,#2509, and#2555(all prerequisites have been merged)