Controller-runtime: Why is a Logger not injected by the Controller Manager?

[akutz]

Hi there,

Along with the other injector-style interfaces, there is InjectLogger, https://github.com/kubernetes-sigs/controller-runtime/blob/57ed2a3ccc943a6a3a07300771326a7e930a9671/pkg/runtime/inject/inject.go#L149-L153

However, the only place this seems to be used is with respect to webhooks.

Why is InjectLogger not part of the SetFields function in the Controller Manager so loggers are added to components added to the manager at https://github.com/kubernetes-sigs/controller-runtime/blob/ea795f6400dc064dce43c0aa6a065f47e0633077/pkg/manager/internal.go#L244-L270

[alvaroaleman]

Looks like an oversight

[vincepri]

/milestone v0.6.x
/kind bug

Can/Should also be backported.

[vincepri]

/help

[k8s-ci-robot]

@vincepri:
This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

/help

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.

[jiachengxu]

Would like to work on this :)
/assign