Skip to content
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

chore(das): improve logging #2209

Merged
merged 4 commits into from
May 16, 2023
Merged

chore(das): improve logging #2209

merged 4 commits into from
May 16, 2023

Conversation

Wondertan
Copy link
Member

@Wondertan Wondertan commented May 15, 2023

Every new head now produces fewer logs + makes the more informative

Looks like below now
image

@Wondertan Wondertan self-assigned this May 15, 2023
@Wondertan Wondertan added area:shares Shares and samples area:das Related to DASer and removed area:shares Shares and samples labels May 15, 2023
das/worker.go Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented May 15, 2023

Codecov Report

Merging #2209 (39090a1) into main (2fa72c7) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2209      +/-   ##
==========================================
- Coverage   55.84%   55.84%   -0.01%     
==========================================
  Files         216      216              
  Lines       14106    14115       +9     
==========================================
+ Hits         7877     7882       +5     
- Misses       5442     5447       +5     
+ Partials      787      786       -1     
Impacted Files Coverage Δ
das/subscriber.go 88.23% <100.00%> (ø)
das/worker.go 95.83% <100.00%> (+0.18%) ⬆️

... and 4 files with indirect coverage changes

distractedm1nd
distractedm1nd previously approved these changes May 16, 2023
@Wondertan Wondertan enabled auto-merge (squash) May 16, 2023 07:55
das/worker.go Outdated Show resolved Hide resolved
distractedm1nd
distractedm1nd previously approved these changes May 16, 2023
@Wondertan Wondertan added the kind:fix Attached to bug-fixing PRs label May 16, 2023
@Wondertan Wondertan disabled auto-merge May 16, 2023 13:38
@Wondertan Wondertan enabled auto-merge (squash) May 16, 2023 13:38
@Wondertan Wondertan merged commit d0db7a9 into main May 16, 2023
@Wondertan Wondertan deleted the hlib/das/improve-logging branch May 16, 2023 14:02
vgonkivs pushed a commit to vgonkivs/celestia-node that referenced this pull request May 22, 2023
Every new head now produces fewer logs + makes the more informative
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:das Related to DASer kind:fix Attached to bug-fixing PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants