-
Notifications
You must be signed in to change notification settings - Fork 7
feat(cli): add dump state tool #891
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
base: master
Are you sure you want to change the base?
Conversation
🧪 Network TestsTo run network tests for this PR, use: gh workflow run network-tests.yml -f pr_number=891Available test options:
Test types: Results will be posted as workflow runs in the Actions tab. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #891 +/- ##
==========================================
+ Coverage 46.27% 54.30% +8.02%
==========================================
Files 360 361 +1
Lines 62584 63056 +472
Branches 62584 63056 +472
==========================================
+ Hits 28961 34240 +5279
+ Misses 32108 26905 -5203
- Partials 1515 1911 +396 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
f4483b9 to
11657bc
Compare
11657bc to
3ba11e5
Compare
1f39588 to
a92e67c
Compare
f7cd958 to
d985030
Compare
0d9c5e5 to
ee79bf8
Compare
ee79bf8 to
c517818
Compare
0ac84c5 to
d0a5ba4
Compare
|
Example, how to use:
|
dd10fe7 to
b417d31
Compare
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.
We should dump only required anchors into files and use them during the test. Not full mempool database. In the commit 1e4355f there is an example how to dump the anchor.
b417d31 to
685beaf
Compare
|
❌ CLI reference check failed in CI. Please run |
26d1ac3 to
ad21336
Compare
950626e to
3408490
Compare
Mododo
left a comment
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.
lgtm!
3408490 to
906a580
Compare
feat(collator): remove test collator manager feat(collator): fix dump path feat(collator): rename dump test, fix test stop on received block feat(collator): improve collation_tests
906a580 to
0cb6ccd
Compare
Pull Request Checklist
NODE CONFIGURATION MODEL CHANGES
[None]
BLOCKCHAIN CONFIGURATION MODEL CHANGES
[None]
COMPATIBILITY
[fully compatible]
SPECIAL DEPLOYMENT ACTIONS
[Not Required]
PERFORMANCE IMPACT
[No impact expected]
TESTS
Unit Tests
[No coverage]
Network Tests
[No coverage]
Manual Tests
Run local nodes, run the one to many script, that generate shard blocks with many internals, stop nodes. Run dump state tool with master block id. Script dumps master and referred shards. Run
test_collation_from_dump, that produces new shard blocks and master block.