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

extend e2e test to cover token renewal #832

Open
clux opened this issue Feb 17, 2022 · 0 comments
Open

extend e2e test to cover token renewal #832

clux opened this issue Feb 17, 2022 · 0 comments
Labels
automation ci and testing related help wanted Not immediately prioritised, please help!

Comments

@clux
Copy link
Member

clux commented Feb 17, 2022

What problem are you trying to solve?

Currently our e2e test only checks that cluster auth works, not whether it continues to work past reload points.
Token reloading was introduced in 0.69.0 and we did not have e2e coverage of tests verifying this causing #829.

Describe the solution you'd like

Would like to ensure that e2e tests catch issues related to in-cluster auth and token renewal.

Extend dapp to wait across renewal points and make an api call after this to ensure things still work.

If nessary, expose a sneaky way to reduce refresh time (see #797), and set it for this e2e test, so that we don't have to wait a full minute just to test this for every PR.

Describe alternatives you've considered

running a larger controller test on ci. would still not catch this particular issue.

Target crate for feature

  • e2e
  • kube-client
@clux clux added automation ci and testing related help wanted Not immediately prioritised, please help! labels Feb 17, 2022
This was referenced Feb 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation ci and testing related help wanted Not immediately prioritised, please help!
Projects
None yet
Development

No branches or pull requests

1 participant