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

Unable to use nodeshift in Katacoda environments #197

Closed
jamesfalkner opened this issue Mar 12, 2018 · 2 comments · Fixed by #198
Closed

Unable to use nodeshift in Katacoda environments #197

jamesfalkner opened this issue Mar 12, 2018 · 2 comments · Fixed by #198
Assignees
@lholmquist
Labels
bug enhancement

Comments

@jamesfalkner
Copy link

I went to upgrade to latest nodeshift in a project that runs in a Katacoda environment, where the configuration file for the oc command is not at ~/.kube/config. It's actually at /openshift.local.config/master/admin.kubeconfig, and I was using the configLocation in nodeshift to point at it. I'm not an expert at kubernetes config locations and don't know if this is a bad practice, but removing this from nodeshift prevents my project from using latest nodeshift as this environment does not place the kube config file in the place that nodeshift now requires.
lholmquist added a commit to lholmquist/nodeshift that referenced this issue Mar 12, 2018
@lholmquist
feat(config-loader): expose the configLocation options for the opensh…
1f67631 
…ift-config-loader.

* This was previously taken out, but is needed again.
* use --configLocation on the cli or the options.configLocation in the API

fixes nodeshift#197
lholmquist added a commit to lholmquist/nodeshift that referenced this issue Mar 12, 2018
@lholmquist
feat(config-loader): expose the configLocation options for the opensh…
73e6285 
…ift-config-loader.

* This was previously taken out, but is needed again.
* use --configLocation on the cli or the options.configLocation in the API

fixes nodeshift#197
@lholmquist lholmquist mentioned this issue Mar 12, 2018
Merged
@ghost ghost assigned lholmquist Mar 12, 2018
@ghost ghost added review and removed in progress labels Mar 12, 2018
lholmquist added a commit that referenced this issue Mar 12, 2018
@lholmquist
feat(config-loader): expose the configLocation options for the opensh…
7462ead 
…ift-config-loader. (#198)

* This was previously taken out, but is needed again.
* use --configLocation on the cli or the options.configLocation in the API

fixes #197
@ghost ghost removed the review label Mar 12, 2018
@jamesfalkner
Copy link
Author

Wahoo!! Once this shows up in a npm-consumable release I'll be sure to remove my workarounds (which right now is a symlink added to ~/.kube :) )

@lholmquist
Copy link
Member

@jamesfalkner i just published it to npm, as 1.5.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lholmquist lholmquist

Labels
bug enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(config-loader): expose the configLocation options for the opensh… lholmquist/nodeshift
2 participants
@jamesfalkner @lholmquist