-
Notifications
You must be signed in to change notification settings - Fork 407
WIP: 🐛 authentication: add custom SA lookup with ttl cache for non-local clusters #3274
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
base: main
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…sters Signed-off-by: Dr. Stefan Schimanski <[email protected]>
c6c34c8
to
82d13e5
Compare
@sttts: The following tests failed, say
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
TODO: this commit may be dropped once kcp-dev/kcp#3274 and related issues are done Signed-off-by: Karol Szwaj <[email protected]> On-behalf-of: @SAP [email protected]
TODO: this commit may be dropped once kcp-dev/kcp#3274 and related issues are done Signed-off-by: Karol Szwaj <[email protected]> On-behalf-of: @SAP [email protected]
TODO: this commit may be dropped once kcp-dev/kcp#3274 and related issues are done Signed-off-by: Karol Szwaj <[email protected]> On-behalf-of: @SAP [email protected]
Summary
With service accounts valid (even as
system:anonymous
+system:authenticated
) on other clusters, we have to validate them on other shards. As there are many service accounts, we cannot distribute them via the cache server. But luckily there are not many those service accounts. We can use some ttl cache for that and use a real external client.TODO
Related issue(s)
Fixes # #3289
Release Notes