-
Notifications
You must be signed in to change notification settings - Fork 135
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
controllers: switch to k8s contextual logger #1308
base: incubation
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 |
/cc @zdtsw |
Jira: https://issues.redhat.com/browse/RHOAIENG-14096 This is a squashed commit of the following patches: d012b67 ("controllers: switch to k8s contextual logger") 9880530 ("logger: blindly convert ctrl.Log users to contextual") - controllers: switch to k8s contextual logger Remove own logger from controllers' reconcilers and switch to k8s contextual logger instead [1]. Use contextual logger for SecretGeneratorReconciler and CertConfigmapGeneratorReconciler setups as well. Add name to the logger coming from the framework. It will contains "controller" field already, and like in webhook with the name it's easy to distinguish framework and operator messages. - logger: blindly convert ctrl.Log users to contextual All the users should have proper context now. The log level changes will affect it as well. [1] https://www.kubernetes.dev/blog/2022/05/25/contextual-logging/ Signed-off-by: Yauheni Kaliuta <[email protected]>
5e678ae
to
7af3c4e
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## incubation #1308 +/- ##
=============================================
Coverage ? 17.50%
=============================================
Files ? 30
Lines ? 3360
Branches ? 0
=============================================
Hits ? 588
Misses ? 2710
Partials ? 62 ☔ View full report in Codecov by Sentry. |
Jira: https://issues.redhat.com/browse/RHOAIENG-14096
This is a squashed commit of the following patches:
d012b67 ("controllers: switch to k8s contextual logger")
9880530 ("logger: blindly convert ctrl.Log users to contextual")
Remove own logger from controllers' reconcilers and switch to k8s contextual logger instead [1].
Use contextual logger for SecretGeneratorReconciler and CertConfigmapGeneratorReconciler setups as well.
Add name to the logger coming from the framework. It will contains "controller" field already, and like in webhook with the name it's easy to distinguish framework and operator messages.
All the users should have proper context now. The log level changes will affect it as well.
[1] https://www.kubernetes.dev/blog/2022/05/25/contextual-logging/
Description
How Has This Been Tested?
Screenshot or short clip
Merge criteria