Skip to content
This repository has been archived by the owner on Aug 24, 2023. It is now read-only.

chore(logger): adopt new logging format #38

Merged
merged 1 commit into from
Jun 9, 2023

Conversation

heiruwu
Copy link
Member

@heiruwu heiruwu commented Jun 9, 2023

Because

  • unify log message structure for better readability and queryability

This commit

  • update log message structure

@linear
Copy link

linear bot commented Jun 9, 2023

@codecov
Copy link

codecov bot commented Jun 9, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (cd50d4a) 12.08% compared to head (36651b3) 12.08%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #38   +/-   ##
=======================================
  Coverage   12.08%   12.08%           
=======================================
  Files           4        4           
  Lines         596      596           
=======================================
  Hits           72       72           
  Misses        520      520           
  Partials        4        4           
Flag Coverage Δ
unittests 12.08% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/service/connector.go 0.00% <0.00%> (ø)
pkg/service/model.go 0.00% <0.00%> (ø)
pkg/service/pipeline.go 0.00% <0.00%> (ø)
pkg/service/service.go 30.25% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@heiruwu heiruwu merged commit 2479254 into main Jun 9, 2023
12 checks passed
@heiruwu heiruwu deleted the heiru/ins-889-adopt-new-logging-format-in-controller branch June 9, 2023 11:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
1 participant