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

Increment Metric at the Start of Validation #4546

Merged
merged 1 commit into from Jan 14, 2020
Merged

Conversation

nisdas
Copy link
Member

@nisdas nisdas commented Jan 14, 2020

To ensure the message received metrics are accurate, we increment whenever we receive any message from gossipsub.

@nisdas nisdas added the Ready For Review A pull request ready for code review label Jan 14, 2020
@codecov
Copy link

codecov bot commented Jan 14, 2020

Codecov Report

Merging #4546 into master will decrease coverage by 16.13%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master    #4546       +/-   ##
===========================================
- Coverage   43.72%   27.58%   -16.14%     
===========================================
  Files         193      192        -1     
  Lines       13359    13016      -343     
===========================================
- Hits         5841     3591     -2250     
- Misses       6513     8795     +2282     
+ Partials     1005      630      -375

@prylabs-bulldozer prylabs-bulldozer bot merged commit 4835ba7 into master Jan 14, 2020
@delete-merged-branch delete-merged-branch bot deleted the shiftMetric branch January 14, 2020 16:49
cryptomental pushed a commit to cryptomental/prysm that referenced this pull request Feb 24, 2020
cryptomental pushed a commit to cryptomental/prysm that referenced this pull request Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review A pull request ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants