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

Periodically refresh ServiceAccount tokens #205

Merged

Conversation

JacobHenner
Copy link
Contributor

Periodically refresh ServiceAccount tokens. This is required to avoid
authentication errors when time-bound tokens 1 are rotated, and the
initially-read token expires. Time bound tokens are beta in Kubernetes
1.21, and GA in 1.22 2.

Copy link
Owner

@tomplus tomplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you rebase your branch, I've fixed e2e tests and should pass now.

kubernetes_asyncio/config/incluster_config.py Outdated Show resolved Hide resolved
kubernetes_asyncio/config/incluster_config.py Outdated Show resolved Hide resolved
kubernetes_asyncio/config/incluster_config.py Outdated Show resolved Hide resolved
Periodically refresh ServiceAccount tokens. This is required to avoid
authentication errors when time-bound tokens [1] are rotated, and the
initially-read token expires. Time bound tokens are beta in Kubernetes
1.21, and GA in 1.22 [2].

[1]: https://github.com/kubernetes/enhancements/tree/master/keps/sig-auth/1205-bound-service-account-tokens
[2]: kubernetes/enhancements#542
@JacobHenner JacobHenner force-pushed the feature/refresh-service-account-tokens branch from b5fc07f to f740775 Compare May 23, 2022 22:32
@codecov
Copy link

codecov bot commented May 23, 2022

Codecov Report

Merging #205 (f740775) into master (af050bb) will increase coverage by 0.03%.
The diff coverage is 95.12%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   27.39%   27.43%   +0.03%     
==========================================
  Files         702      702              
  Lines       93973    94004      +31     
==========================================
+ Hits        25748    25787      +39     
+ Misses      68225    68217       -8     
Impacted Files Coverage Δ
kubernetes_asyncio/config/incluster_config.py 92.18% <91.66%> (+4.43%) ⬆️
kubernetes_asyncio/config/incluster_config_test.py 98.00% <100.00%> (+0.38%) ⬆️
kubernetes_asyncio/client/configuration.py 72.67% <0.00%> (+3.82%) ⬆️

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 af050bb...f740775. Read the comment docs.

@JacobHenner
Copy link
Contributor Author

Rebased, requested changes made.

@tomplus
Copy link
Owner

tomplus commented May 24, 2022

Thanks for your contribution.

@tomplus tomplus merged commit 50fc5f8 into tomplus:master May 24, 2022
tomplus pushed a commit that referenced this pull request May 24, 2022
Periodically refresh ServiceAccount tokens. This is required to avoid
authentication errors when time-bound tokens [1] are rotated, and the
initially-read token expires. Time bound tokens are beta in Kubernetes
1.21, and GA in 1.22 [2].

[1]: https://github.com/kubernetes/enhancements/tree/master/keps/sig-auth/1205-bound-service-account-tokens
[2]: kubernetes/enhancements#542
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

Successfully merging this pull request may close these issues.

2 participants