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.

Sign up for GitHub

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

Jump to bottom

cli tool doesn't respect $KUBECONFIG file location #17

Closed
DylanGraham opened this issue Dec 7, 2017 · 1 comment
Closed

cli tool doesn't respect $KUBECONFIG file location #17

DylanGraham opened this issue Dec 7, 2017 · 1 comment
Labels
area/cli

Comments

@DylanGraham
Copy link

For example:

$ conduit stat pod
stat .kube/config: no such file or directory
@klingerf
Copy link
Member

klingerf commented Dec 7, 2017

Thanks for reporting @DylanGraham. That was an oversight on our part. To work around this for now, you can explicitly include the --kubeconfig flag, but we'll update to support the env variable as well.

pcalcado pushed a commit that referenced this issue Dec 19, 2017
Make k8s API code respect /Users/pcalcado/.kube/config (closes #17)
a12cd76
@pcalcado pcalcado mentioned this issue Dec 19, 2017
Merged
pcalcado pushed a commit that referenced this issue Dec 19, 2017
Make k8s API code respect /Users/pcalcado/.kube/config (closes #17)
ff53a10
pcalcado pushed a commit that referenced this issue Dec 19, 2017
Make k8s API code respect /Users/pcalcado/.kube/config (closes #17)
af761b4
klingerf pushed a commit that referenced this issue Dec 19, 2017
@klingerf
Make k8s API code respect /Users/pcalcado/.kube/config (closes #17)
5eec58e
pcalcado pushed a commit that referenced this issue Dec 19, 2017
Make k8s API code respect /Users/pcalcado/.kube/config (closes #17)
e91ad2a
pcalcado pushed a commit that referenced this issue Dec 19, 2017
Make k8s API code respect /Users/pcalcado/.kube/config (closes #17)
f35c57d
pcalcado pushed a commit that referenced this issue Dec 20, 2017
Make k8s API code respect /Users/pcalcado/.kube/config (closes #17)
266431d
pcalcado added a commit that referenced this issue Dec 20, 2017
@pcalcado
Respect $KUBECONFIG env var (#68)
0a6a9ed 
* Move kubectl logis to k8s package

* Made kubectl return *url.URL, just like API

* Make k8s API code respect /Users/pcalcado/.kube/config (closes #17)

* Fix style mistakes and typos
@klingerf klingerf mentioned this issue Feb 1, 2018
Merged
khappucino pushed a commit to Nordstrom/linkerd2 that referenced this issue Mar 5, 2019
@briansmith
Replace "conduit" with "linkerd" in TLS test data. (linkerd#17)
1fdcbfa 
This is purely aesthetic; the TLS logic doesn't care about the product name.

The test data was regenerated by re-running gen-certs.sh after modifying it.

Signed-off-by: Brian Smith <brian@briansmith.org>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area/cli
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@pcalcado @klingerf @DylanGraham