Skip to content

Initialize controller-runtime logger in operator#3982

Draft
jhrozek wants to merge 1 commit intomainfrom
operator-zap-init
Draft

Initialize controller-runtime logger in operator#3982
jhrozek wants to merge 1 commit intomainfrom
operator-zap-init

Conversation

@jhrozek
Copy link
Contributor

@jhrozek jhrozek commented Mar 3, 2026

The controller-runtime logger was never initialized, so it used a no-op logger by default and all operator log output was silently discarded.

The controller-runtime logger was never initialized, so it used a no-op
logger by default and all operator log output was silently discarded.
@github-actions github-actions bot added the size/XS Extra small PR: < 100 lines changed label Mar 3, 2026
@jhrozek jhrozek marked this pull request as draft March 3, 2026 22:42
@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 68.53%. Comparing base (c46ae47) to head (e4cdbe6).

Files with missing lines Patch % Lines
cmd/thv-operator/main.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3982      +/-   ##
==========================================
- Coverage   68.56%   68.53%   -0.03%     
==========================================
  Files         437      437              
  Lines       44662    44663       +1     
==========================================
- Hits        30621    30611      -10     
- Misses      11657    11674      +17     
+ Partials     2384     2378       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Extra small PR: < 100 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants