-
Notifications
You must be signed in to change notification settings - Fork 398
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
✨ Switch to cluster-aware k8s clients, listers & informers #2104
Merged
ncdc
merged 27 commits into
kcp-dev:main
from
stevekuznetsov:skuznets/clients-listers-informers
Oct 21, 2022
Merged
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
f66c731
bootstrap: improve logging
stevekuznetsov a5cbd37
*: move our cluster-aware-key hack to our repo
stevekuznetsov e4a3021
test/e2e: improve logging in a fixture
stevekuznetsov 480aaab
admission: implement a delegating limit-ranger plugin
stevekuznetsov 2f16495
admission/kubequota: implement using generated factories
stevekuznetsov cd678ab
builtins: remove AP&F types
stevekuznetsov c8282c1
docs: update key section
stevekuznetsov 96a37b0
admission/*webhook: reimplement with cluster-aware delegation
stevekuznetsov 577fa50
authorization: use scoped listers
stevekuznetsov c10556e
workspace_content_authorizer: fill indexers using client in tests
stevekuznetsov b07cfb3
workspace_content_authorizer: actually name objects correctly
stevekuznetsov edd323b
admission/namespacelifecycle: feed the cluster-aware namespace admission
stevekuznetsov 8422002
server/cache: implement a cluster-aware CRD lister
stevekuznetsov 21fcd85
server: implement a cluster-aware apibinding-aware crd lister
stevekuznetsov fcd70eb
reconciler/cache: make tests cluster-aware
stevekuznetsov 5a50dc4
server: consume the informers + client from the first creation
stevekuznetsov 0ab0550
syncer: remove the scoping lister implementation
stevekuznetsov b12141e
virtual/framework: remove a dynamic cluster client implementation
stevekuznetsov ba7de6f
dynamicsharedinformerfactory: simplify by using generated code
stevekuznetsov 5f4277d
virtual/framework: adapt client getter for new types
stevekuznetsov cd96188
virtual/framework: adapt deleter implemenetation
stevekuznetsov 849e26d
virtual/framework: adapt tests to be cluster-aware
stevekuznetsov d001afd
virtual/workspaces: adapt plumbing of auth cache
stevekuznetsov be2357b
test/e2e/reconciler/namespace: adapt expecter to do a cross-cluster list
stevekuznetsov 2b1d382
*: mostly-automated, boring type/callsite refactors
stevekuznetsov baaded5
*: miscellany to pass linters and reiew comments
stevekuznetsov c948a85
go.mod: bump k8s to latest commit
stevekuznetsov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume you'll drop/replace this commit?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that's the last commit & will be replaced with the fork once we merge the k8s PR