Skip to content
This repository has been archived by the owner on Mar 13, 2022. It is now read-only.

Fix issue when refreshing service account token #40

Merged
merged 1 commit into from
Jan 10, 2018

Conversation

jdartigalongue
Copy link
Contributor

@mbohlool I believe this will fix the below issues:
kubernetes-client/python#339
kubernetes-client/python#386

I was able to reproduce this issue by either setting GOOGLE_APPLICATION_CREDENTIALS to a service account json key file or by setting the gcloud default application login to a service account.

Unlike user accounts, it seems that service accounts require that we set the scope.

@codecov-io
Copy link

codecov-io commented Jan 1, 2018

Codecov Report

Merging #40 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #40   +/-   ##
=======================================
  Coverage   93.36%   93.36%           
=======================================
  Files          11       11           
  Lines         829      829           
=======================================
  Hits          774      774           
  Misses         55       55
Impacted Files Coverage Δ
config/kube_config.py 91.86% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9213876...b7b6e65. Read the comment docs.

@jdartigalongue
Copy link
Contributor Author

jdartigalongue commented Jan 1, 2018

Bump to verify CLA

@mbohlool mbohlool merged commit 13aa7cd into kubernetes-client:master Jan 10, 2018
@mbohlool
Copy link
Contributor

Thanks for your contribution. You also need to send a PR to the main repo updating this sub-module.

@michrabi
Copy link

michrabi commented Jan 17, 2018

How can I get this fix? I tried cloning and I don't see it.
Thanks

@pokoli
Copy link
Contributor

pokoli commented Jan 17, 2018

@michrabi you should create a PR to the main repo (https://github.com/kubernetes-client/python) updating the submodule to include this commit. See: https://github.com/kubernetes-client/python/blob/master/devel/submodules.md

Thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants