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

Kubctl plugin redisoperator didn't work on GKE cluster #14

Closed
clamoriniere1A opened this issue Jul 3, 2018 · 0 comments · Fixed by #15
Closed

Kubctl plugin redisoperator didn't work on GKE cluster #14

clamoriniere1A opened this issue Jul 3, 2018 · 0 comments · Fixed by #15
Assignees
@clamoriniere1A
Labels
bug Something isn't working

Comments

@clamoriniere1A
Copy link
Contributor

clamoriniere1A commented Jul 3, 2018 •
edited
Loading

During a test on GKE cluster, I discovered that kubectl rediscluster plugin didn't works properly with the GKE kubeconfig auth_provider config.

plugin version: git commit_id 695c499

> plugin rediscluster
F0703 18:00:47.723112   20285 main.go:60] Unable to init redis.clientset from kubeconfig:No Auth Provider found for name "gcp"
goroutine 1 [running]:
github.com/amadeusitgroup/redis-operator/vendor/github.com/golang/glog.stacks(0xc42033cc00, 0xc4201481c0, 0x7f, 0xd4)
	/Users/clamoriniere/dev/projects/redis-operator-oss/src/github.com/amadeusitgroup/redis-operator/vendor/github.com/golang/glog/glog.go:766 +0xcf
github.com/amadeusitgroup/redis-operator/vendor/github.com/golang/glog.(*loggingT).output(0x228ede0, 0xc400000003, 0xc42021a2c0, 0x21feb1c, 0x7, 0x3c, 0x0)
	/Users/clamoriniere/dev/projects/redis-operator-oss/src/github.com/amadeusitgroup/redis-operator/vendor/github.com/golang/glog/glog.go:717 +0x30f
github.com/amadeusitgroup/redis-operator/vendor/github.com/golang/glog.(*loggingT).printf(0x228ede0, 0x3, 0x1b8590e, 0x31, 0xc420419890, 0x1, 0x1)
	/Users/clamoriniere/dev/projects/redis-operator-oss/src/github.com/amadeusitgroup/redis-operator/vendor/github.com/golang/glog/glog.go:655 +0x14b
github.com/amadeusitgroup/redis-operator/vendor/github.com/golang/glog.Fatalf(0x1b8590e, 0x31, 0xc420419890, 0x1, 0x1)
	/Users/clamoriniere/dev/projects/redis-operator-oss/src/github.com/amadeusitgroup/redis-operator/vendor/github.com/golang/glog/glog.go:1145 +0x67
main.main()
	/Users/clamoriniere/dev/projects/redis-operator-oss/src/github.com/amadeusitgroup/redis-operator/cmd/kubectl-plugin/main.go:60 +0x427
error: exit status 255

it seams related to: kubernetes/client-go#242
@clamoriniere1A clamoriniere1A added the bug Something isn't working label Jul 3, 2018
@clamoriniere1A clamoriniere1A self-assigned this Jul 3, 2018
clamoriniere1A added a commit to clamoriniere1A/Redis-Operator that referenced this issue Jul 3, 2018
@clamoriniere1A
fix rediscluster kubectl plugin
7b6f122 
fixes: AmadeusITGroup#14

add missing client-go import in order to support auth provider in the
kubectl rediscluster plugin.
Also improve the way the plugin process retrieve the kubeconfig file
path.
clamoriniere1A added a commit to clamoriniere1A/Redis-Operator that referenced this issue Jul 3, 2018
@clamoriniere1A
fix rediscluster kubectl plugin
479f296 
fixes: AmadeusITGroup#14

add missing client-go import in order to support auth provider in the
kubectl rediscluster plugin.
Also improve the way the plugin process retrieve the kubeconfig file
path.
@clamoriniere1A clamoriniere1A mentioned this issue Jul 3, 2018
Merged
clamoriniere1A added a commit to clamoriniere1A/Redis-Operator that referenced this issue Jul 4, 2018
@clamoriniere1A
fix rediscluster kubectl plugin
ce30e74 
fixes: AmadeusITGroup#14

- add missing client-go import in order to support auth provider in the
kubectl rediscluster plugin.
- improve the way the plugin process retrieve the kubeconfig file
path.
- improve error handling in kubectl plugin
- fix bug in operator after dependency revendoring
clamoriniere1A added a commit that referenced this issue Jul 4, 2018
@clamoriniere1A
fix rediscluster kubectl plugin (#15)
68f754e 
fixes: #14

- add missing client-go import in order to support auth provider in the
kubectl rediscluster plugin.
- improve the way the plugin process retrieve the kubeconfig file
path.
- improve error handling in kubectl plugin
- fix bug in operator after dependency revendoring
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@clamoriniere1A clamoriniere1A

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix rediscluster kubectl plugin clamoriniere1A/Redis-Operator
1 participant
@clamoriniere1A