Closed coolbrg closed 8 years ago
I've left two comments inline that can be turned into issues and should not block this PR.
However, I don't see the code that creates the config file for kubectl as referenced in https://gist.github.com/praveenkumar/90704a42fe857e1146f3dae2fcae69ef
Am I missing it?
@budhrg you mentioned in IRC that kube config creation is part of a separate PR. Where is that PR? I believe this code needs to be dependent on kubectl working once downloaded/installed.
@bexelbie It is not dependent on the other PR since its an individual feature in itself.
@budhrg I agree it is not dependent in a technical sense. However, from a user perspective we either need to tell them how to build their own config file in documentation or we need to add the code that generates the config and installs it for them. I am suggesting the dependency is usability based.
@bexelbie , @navidshaikh : I have updated the PR with another commit to configure kube config and showing its path through env kubernetes
.
Note: To verify this patch, one need to use this patched box.
User workflow would be :
vagrant up
. It will create a kubeconfig
file in ~/.vagrant.d/data/service-manager/kubeconfig
path containing required details like token and server ipeval "$(vagrant service-manager env kubernetes)"
kubectl
as eval "$(VAGRANT_NO_COLOR=1 vagrant service-manager install-cli kubernetes | tr -d '\r')"
kubectl version
retest this please
The code visually reviews well. I do not have time to test it today, unfortunately. @navidshaikh @tkral or @praveenkumar can you get to it today/your tomorrow morning?
@budhrg I tested with latest CDK and kubeconf generated as expected and also binary was installed without any issue. LGTM
Let's wait for @navidshaikh test.
Working in Windows 10 too.
Working for Mac and Linux too.
LGTM
@navidshaikh / @bexelbie Can you merge it if that is working as expected?
Fix #350
Also includes:
config.servicemanager.services = 'kubernetes
For quick test at local one can use
bundle exec rake test rubocop features
.My logs: