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

Global config should only support local-context on a per kube-context basis #4554

Open
briandealwis opened this issue Jul 23, 2020 · 1 comment
Labels
area/global-config kind/friction Issues causing user pain that do not have a workaround priority/p1 High impact feature/bug.

Comments

@briandealwis
Copy link
Member

@russwolf somehow had his ~/.skaffold/config sert to make all cluster be non-local which led to very puzzling errors when attempting to develop against local clusters. He’s unsure how this setting was configured. And this particular setting seems of questionable value as a blanket setting, and should instead be set on a per-kube-context basis.

We should revisit the actual global config definition as it allows invalid and questionable configuration combinations. For example, it allows setting insecure registries and debug helper locations on a per-kube-context basis.

@briandealwis briandealwis added area/global-config priority/p1 High impact feature/bug. kind/friction Issues causing user pain that do not have a workaround labels Jul 23, 2020
@etanshaul
Copy link
Contributor

for the IDEs, can Skaffold surface a better error message in this case (where the cluster is configured as non-local, but the user has no default-registry set)?

@nkubala nkubala added this to the Backlog milestone Sep 1, 2020
@nkubala nkubala removed this from the Backlog [P0/P1] milestone May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/global-config kind/friction Issues causing user pain that do not have a workaround priority/p1 High impact feature/bug.
Projects
None yet
Development

No branches or pull requests

3 participants