-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
fix: log errors setting agent version as debug #6374
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6374 +/- ##
============================================
+ Coverage 60.14% 60.16% +0.01%
============================================
Files 407 407
Lines 46486 46514 +28
Branches 1545 1550 +5
============================================
+ Hits 27957 27983 +26
- Misses 18497 18499 +2
Partials 32 32 |
Performance Report✔️ no performance regression detected 🚀🚀 Significant benchmark improvement detected
Full benchmark results
|
Co-authored-by: Nico Flaig <nflaig@protonmail.com>
there are so great benchmark statistics in this PR although there's nothing to do with it. I think at least we need to switch validator size to 1.5M instead of 250k for a lot of tests. |
LOL!!! Nope... that was me just having "the 🥷 touch" like you do @tuyennhv .... those are all REAL i tell you!! ARE EEE AYE ELLE REAL!!!! OK maybe you are right.... |
🎉 This PR is included in v1.16.0 🎉 |
Motivation
Seeing an error when attempting to setAgent for a peer that is no longer in the peerStore
@wemeetagain @tuyennhv Do you think this error should be skipped because its just a result of peer churn and the info already being removed from the DB or do you think its worth making it a
debug
statement?Resolves #6365
Description
Log errors setting agent version as debug
Add check for leveldbNotFound
error and do not log as its an anticipated error.NOTE: Double checking the leveldb code where this originates to see if there is a better way to filter for the error. Will move to readyToReveiw once complete