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
Remove dependency injection functions #8217
Conversation
Skipping CI for Draft Pull Request. |
/test all |
08c5f68
to
aa855e7
Compare
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.
Thanks a lot for the very well structured PR.
I just have a minor question. Apart from that it looks good already.
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.
Thanks for the well-structured PR.
extensions/pkg/controller/controlplane/genericactuator/actuator.go
Outdated
Show resolved
Hide resolved
/retest |
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.
/lgtm
LGTM label has been added. Git tree hash: 7a09b7c7a77db56743ea2ffb8ab2bd70aba9da86
|
/assign |
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.
Nice work, thanks for the change!
I'm not through with my review, but here are some comments from the first pass.
extensions/pkg/controller/controlplane/genericactuator/actuator.go
Outdated
Show resolved
Hide resolved
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.
Made it through, here are the remaining comments :)
…eRequestsFrom` function
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.
Awesome, nice work @ary1992!
/lgtm
/approve
|
||
// ClientContext bundles the feature of providing injected scheme and client for | ||
// the controller runtime. Additionally it offers a decoder using the scheme. | ||
type ClientContext struct { |
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.
Can you mention/explain the removal of this package in the release note as well? :)
LGTM label has been added. Git tree hash: fa1059deb98598618b2d055d84c5568c55f22b9e
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: timebertt The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
* Adapt extensions/pkg/controller/backupBucket * Adapt extensions/pkg/controller/backupEntry * Adapt extensions/pkg/controller/containerruntime * Adapt extensions/pkg/controller/controlplane * Adapt extensions/pkg/controller/dnsrecord * Adapt extensions/pkg/controller/extension * Adapt extensions/pkg/controller/healthcheck * Adapt extensions/pkg/controller/network * Adapt extensions/pkg/controller/bastion * Adapt extensions/pkg/controller/infrastructure * Adapt extensions/pkg/controller/operatingsystemconfig * Adapt extensions/pkg/controller/controlplane * Adapt pkg/controllerutils/reconciler * Adapt extensions/pkg/controller/healthcheck * Adapt extensions/pkg/controller/heartbeat * Adapt extensions/pkg/controller/worker * Adapt provider-local * Add `context.Context` as first argument to AddToManager function * Adapt extensions/pkg/predicate/ * Adapt validator, handler and mutator * Adapt `ClientContext` for extensions controller * Adapt pkg/gardenlet/controller/controllerinstallation * Adapt pkg/gardenlet/controller/managedseed * Adapt pkg/controllermanager/controller/managedseedset * Adapt pkg/controllerutils/mapper * Adapt extensions/pkg/controller/mock/ * Adapt tests * [OperationAnnotationWrapper] Address PR review * Address PR review feedback * Drop pkg/controller/common * [Address PR review feedback] Pass cache instead of manager to `EnqueueRequestsFrom` function
How to categorize this PR?
/area open-source dev-productivity
/kind enhancement
What this PR does / why we need it:
The package
controller-runtime/pkg/runtime/inject
has been removed incontroller-runtime
v0.15
. This package contained long deprecated injection functions (likeInjectScheme
,InjectLogger
,InjectConfig
,InjectClient
,InjectCache
, etc.).The runtime injection functionality has been deprecated since
Controller Runtime 0.10
; all of the above fields can be passed from theManager
to structs or interfaces that need them.kubernetes-sigs/controller-runtime#2134
kubernetes-sigs/controller-runtime#2120
Which issue(s) this PR fixes:
Part of #7991
Special notes for your reviewer:
Release note: