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

Image kubectl not found because of Kubernetes version #266

Closed
galexrt opened this issue Jan 3, 2018 · 4 comments
Closed

Image kubectl not found because of Kubernetes version #266

galexrt opened this issue Jan 3, 2018 · 4 comments

Comments

@galexrt
Copy link
Contributor

galexrt commented Jan 3, 2018

The operator fails to start when the Kubernetes version looks like this:

Server Version: version.Info{Major:"1", Minor:"8+", GitVersion:"v1.8.5+coreos.0", GitCommit:"b8e596026feda7b97f4337b115d1a9a250afa8ac", GitTreeState:"clean", BuildDate:"2017-12-12T11:01:08Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

This fails because of the + in the minor version part.
Code ref: https://github.com/appscode/stash/blob/aa91dadd1ac97792a19e6d5470a6f80acdbb9e20/pkg/cmds/run.go#L61

Operator logs:

W0103 08:45:04.001676       1 client_config.go:529] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
F0103 08:45:05.073765       1 run.go:63] Image appscode/kubectl:1.8+.0 not found.

A fix would be to remove every non-numeric char from the minor version string.
What do you think should be changed here so it is fixed?

@tamalsaha
Copy link
Member

tamalsaha commented Jan 3, 2018

Fixed here: ac4e8c9

@tamalsaha
Copy link
Member

Just noticed that. in GKE:

 $ kubectl version 
Client Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.3", GitCommit:"f0efb3cb883751c5ffdbe6d515f3cb4fbe7b7acd", GitTreeState:"clean", BuildDate:"2017-11-08T18:39:33Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"8+", GitVersion:"v1.8.4-gke.1", GitCommit:"04502ae78d522a3d410de3710e1550cfb16dad4a", GitTreeState:"clean", BuildDate:"2017-12-08T17:24:53Z", GoVersion:"go1.8.3b4", Compiler:"gc", Platform:"linux/amd64"}

@tamalsaha
Copy link
Member

Fixed in 0.6.1

@tamalsaha
Copy link
Member

Thanks for the quick error report :)

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

No branches or pull requests

2 participants