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

feat(syncer/metrics): add metrics to syncer #136

Merged
merged 21 commits into from
Feb 1, 2024

Conversation

vgonkivs
Copy link
Member

Overview

Add metrics to syncer

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@vgonkivs vgonkivs self-assigned this Dec 13, 2023
@vgonkivs vgonkivs force-pushed the metrics_for_syncer branch 2 times, most recently from 0b7da9e to bd7bb1f Compare December 13, 2023 11:30
@codecov-commenter
Copy link

codecov-commenter commented Dec 13, 2023

Codecov Report

Attention: 75 lines in your changes are missing coverage. Please review.

Comparison is base (28ff21c) 63.60% compared to head (0070afe) 63.03%.
Report is 13 commits behind head on main.

Files Patch % Lines
sync/metrics.go 11.90% 69 Missing and 5 partials ⚠️
sync/sync_head.go 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #136      +/-   ##
==========================================
- Coverage   63.60%   63.03%   -0.58%     
==========================================
  Files          39       39              
  Lines        3366     3476     +110     
==========================================
+ Hits         2141     2191      +50     
- Misses       1060     1113      +53     
- Partials      165      172       +7     

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

sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
@vgonkivs
Copy link
Member Author

vgonkivs commented Dec 15, 2023

Снимок экрана 2023-12-15 в 12 47 45
Снимок экрана 2023-12-15 в 12 46 04

Please note, that peer info is not shown on the second screenshot because only one node was running

@vgonkivs vgonkivs marked this pull request as ready for review December 15, 2023 10:50
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/sync_head.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/sync.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/sync_head.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/sync_head.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
@vgonkivs vgonkivs requested review from renaynay and walldiss January 10, 2024 15:48
sync/metrics.go Outdated Show resolved Hide resolved
sync/sync_head.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
@Wondertan
Copy link
Member

@vgonkivs, this is a feature, not misc

sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
sync/sync_head.go Outdated Show resolved Hide resolved
sync/sync_head.go Outdated Show resolved Hide resolved
sync/sync_head.go Outdated Show resolved Hide resolved
Copy link
Member

@Wondertan Wondertan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopingly last bits

sync/metrics.go Outdated Show resolved Hide resolved
sync/sync_head.go Outdated Show resolved Hide resolved
sync/metrics.go Outdated Show resolved Hide resolved
@Wondertan Wondertan changed the title misc(syncer/metrics): add metrics to syncer feat(syncer/metrics): add metrics to syncer Jan 31, 2024
@vgonkivs vgonkivs dismissed stale reviews from Wondertan and walldiss via b3f5719 February 1, 2024 10:02
@vgonkivs vgonkivs enabled auto-merge (squash) February 1, 2024 12:05
@vgonkivs vgonkivs dismissed renaynay’s stale review February 1, 2024 13:33

need to ship in order to start testing

@vgonkivs vgonkivs merged commit 30ce8bc into celestiaorg:main Feb 1, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants