-
-
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
feat: allow to skip gossip validation #6308
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6308 +/- ##
=========================================
Coverage 80.38% 80.39%
=========================================
Files 202 202
Lines 19620 19628 +8
Branches 1176 1176
=========================================
+ Hits 15771 15779 +8
Misses 3821 3821
Partials 28 28 |
Performance Report✔️ no performance regression detected Full benchmark results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i can't see you using this flag to setup validator on the tests you are targeting to fix?
I will use this in other PR which is ready and need this one to merge first. |
* Allow to skip gossip validation * Add code comment
🎉 This PR is included in v1.15.0 🎉 |
Motivation
Make sure the block is published to network.
Description
In case a node is configured only with the
unknownBlockSync
, it needs to know the unknown parent blocks on the network to initiate the syncing process. Such cases can be covered only if we publish blocks and make sure no gossip validation is performed on those. But this behavior is not the default.Steps to test or reproduce