Skip to content

Adjust log level and fix log format#1270

Open
yanjunz97 wants to merge 1 commit intovmware-tanzu:mainfrom
yanjunz97:log-warn-error
Open

Adjust log level and fix log format#1270
yanjunz97 wants to merge 1 commit intovmware-tanzu:mainfrom
yanjunz97:log-warn-error

Conversation

@yanjunz97
Copy link
Contributor

No description provided.

@yanjunz97 yanjunz97 force-pushed the log-warn-error branch 2 times, most recently from f0301cd to c765814 Compare October 24, 2025 06:24
@codecov-commenter
Copy link

codecov-commenter commented Oct 24, 2025

Codecov Report

❌ Patch coverage is 61.81818% with 84 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.59%. Comparing base (86a8839) to head (c2e96a4).

Files with missing lines Patch % Lines
...kg/controllers/subnetport/subnetport_controller.go 39.13% 12 Missing and 2 partials ⚠️
pkg/controllers/networkinfo/networkinfo_utils.go 50.00% 7 Missing and 2 partials ⚠️
...ervices/ipaddressallocation/ipaddressallocation.go 25.00% 9 Missing ⚠️
pkg/nsx/services/subnetport/subnetport.go 68.18% 7 Missing ⚠️
pkg/controllers/node/node_controller.go 40.00% 6 Missing ⚠️
pkg/nsx/services/nsxserviceaccount/cluster.go 14.28% 6 Missing ⚠️
pkg/controllers/namespace/namespace_controller.go 33.33% 2 Missing and 2 partials ⚠️
pkg/controllers/pod/pod_controller.go 40.00% 3 Missing ⚠️
pkg/nsx/services/ipblocksinfo/ipblocksinfo.go 85.00% 3 Missing ⚠️
...ddressallocation/ipaddressallocation_controller.go 50.00% 2 Missing ⚠️
... and 15 more

❌ Your patch status has failed because the patch coverage (61.81%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1270      +/-   ##
==========================================
- Coverage   75.63%   75.59%   -0.05%     
==========================================
  Files         148      148              
  Lines       19875    19915      +40     
==========================================
+ Hits        15033    15055      +22     
- Misses       3746     3758      +12     
- Partials     1096     1102       +6     
Flag Coverage Δ
unit-tests 75.59% <61.81%> (-0.05%) ⬇️
Files with missing lines Coverage Δ
pkg/config/config.go 70.81% <100.00%> (ø)
pkg/controllers/common/utils.go 81.93% <100.00%> (-0.12%) ⬇️
.../controllers/networkinfo/networkinfo_controller.go 72.74% <100.00%> (ø)
...ontrollers/networkinfo/vpcnetworkconfig_handler.go 56.36% <100.00%> (-3.64%) ⬇️
pkg/controllers/subnet/subnet_poll.go 71.13% <100.00%> (ø)
pkg/controllers/subnet/subnet_webhook.go 86.84% <100.00%> (ø)
...trollers/subnetbinding/subnetbinding_controller.go 76.60% <100.00%> (+0.22%) ⬆️
...g/controllers/subnetport/addressbinding_webhook.go 100.00% <100.00%> (ø)
pkg/controllers/subnetset/namespace_handler.go 100.00% <100.00%> (ø)
pkg/nsx/client.go 86.76% <100.00%> (-0.07%) ⬇️
... and 29 more

... and 1 file with indirect coverage changes

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

@yanjunz97 yanjunz97 force-pushed the log-warn-error branch 4 times, most recently from 5e061f1 to b5dec3e Compare October 28, 2025 03:02
TaoZou1
TaoZou1 previously approved these changes Nov 3, 2025
Signed-off-by: Yanjun Zhou <yanjun.zhou@broadcom.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants