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

fix(kube-config): use KUBECONFIG env var if set #322

Merged
merged 1 commit into from
Sep 6, 2018
Merged

fix(kube-config): use KUBECONFIG env var if set #322

merged 1 commit into from
Sep 6, 2018

Conversation

JacopoDaeli
Copy link
Contributor

@JacopoDaeli JacopoDaeli commented Sep 4, 2018

@jcrugzz
Copy link
Collaborator

jcrugzz commented Sep 5, 2018

@JacopoDaeli It shouldn't be more than a deep merge right once converting to JSON? Seems reasonable to support it. Doesn't have to be this PR though

@JacopoDaeli
Copy link
Contributor Author

Yeah I agree with it. I am happy to work on a separate PR to support it.

@silasbw
Copy link
Contributor

silasbw commented Sep 5, 2018

Should we go straight to supporting the merge? What do we get by supporting the first kubeconfig file in the path? That seems like it's an unexpected thing to support.

@JacopoDaeli
Copy link
Contributor Author

JacopoDaeli commented Sep 5, 2018

I think "having one single kubeconfig file" is the most common case, and with this PR covers it. I see it like a mid step. However, I am ok with supporting the merge right away. Whatever u folks feel better. I have no strong opinion on this.

@silasbw
Copy link
Contributor

silasbw commented Sep 6, 2018

OK, sounds good to me.

@silasbw silasbw merged commit c6810de into godaddy:master Sep 6, 2018
@JacopoDaeli JacopoDaeli deleted the jdaeli/fixEnvKubeVar0 branch September 6, 2018 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants