-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Introduce basic slot-based collator #4097
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
85 commits
Select commit
Hold shift + click to select a range
9248cd1
Refactor block building and import in collator service
skunert 70afa25
Remove unused SyncOracle, light reformat
skunert 7e01dca
Add slot-based collator
skunert 08ec3e7
Prepare cumulus test node & runtime
skunert 3848239
Prepare cumulus test node & runtime
skunert 7a057ed
Add zombienet test
skunert 3f4a260
Misc
skunert c049f50
Nits
skunert c1c96bc
Remove unused dep, add issues to todos
skunert 7af20a6
Remove unused dep, add issues to todos
skunert 245a11a
Enable "elastic-scaling-experimental" for cumulus test node
skunert 560a907
Adjust parent search to allow for pending blocks with depth > 1
skunert 734bdcb
Merge fixes
skunert c439d21
Modernize zombienet test:
skunert fb9f383
Unify core schedule fetching methods
skunert 06f8fc7
Remove overseer_handle from parameters
skunert cf64247
Improve logging
skunert aa7fba9
Merge branch 'master' into slot-based-mvp
skunert 44b489f
Do not return potential_parents that are not available locally
skunert 8df558f
Use commong async backing params fetching methods
skunert 7705314
Assign cores during block building
skunert 1553d40
Merge branch 'master' into slot-based-mvp
skunert f84a6f3
Add more comments, cleanup
skunert e1a5879
Refactor `find_potential_parent`
skunert 576605b
Merge branch 'master' into slot-based-mvp
skunert 1b184da
Make expected cores work when para slot duration > relay slot duration
skunert e95e5df
Review comments
skunert e4edbe4
Extract parent search into module
skunert 53869a0
More code reuse between lookahead and slot-based
skunert 4240cc1
Merge branch 'master' into slot-based-mvp
skunert c790fb8
fmt
skunert 61be48b
Merge branch 'master' into slot-based-mvp
skunert a06bf79
Merge branch 'master' into slot-based-mvp
skunert 169479a
Add slot drift for testing
skunert ac3e9ed
Merge branch 'master' into slot-based-mvp
skunert 6a9955d
Remove duplicate comment
skunert 9760e81
Expose slot_drift parameter
skunert 42c0135
Do not assumed fixed slot_duration
skunert 44bc8e7
Apply suggestions from code review
skunert a23d5a9
Address comments
skunert 07478dc
Remove unused parameters from zombienet
skunert 15c36a9
Introduce experimental CLI option
skunert 2c8cc0d
Merge branch 'master' into slot-based-mvp
skunert 65cdc01
fmt
skunert 5dcea6c
Adjust zombienet parameter
skunert 46bd385
Merge branch 'master' into slot-based-mvp
skunert 27eb643
Remove +1 on velocity
skunert 99e5741
Reduce number of relay chain fetches
skunert 944044d
Add comments to `RelayChainCachingFetcher`
skunert 41dba02
More comment adjustments
skunert 33243a2
Merge branch 'master' into slot-based-mvp
skunert 0b061f6
Fix template build
skunert 8be5db4
Apply suggestions from code review
skunert 8e4320d
Merge branch 'master' into slot-based-mvp
skunert 6c97bd1
Merge option in relay chain fetcher
skunert 0ae68d3
Spawn futures inside collator
skunert 767430e
Do not build if pending parent is not in db
skunert 1021ee3
Remove slot based experimental from common cli options
skunert c109088
Fix lookahead collator filling the pipeline
skunert 8c364e8
Add PoV-recovery test, make sure no inherent errors are thrown
skunert 8e30d38
Move collator to futures again
skunert f3233db
Move lost comment
skunert da96123
Improve pov-recovery test
skunert 05c3812
Increase velocity to 4
skunert 938a736
Add test to pipeline
skunert fa3f070
Add prdoc
skunert edfe474
Abort block builder task if we can not send to collation task
skunert b1490eb
Fixes
skunert 17fd978
Merge branch 'master' into slot-based-mvp
skunert 9d14989
Comment adjustments
skunert 96d885a
Fix pov-recovery test
skunert eb61449
Fix prdoc semver
skunert cb16053
Add slot based collator for default runtimes or chainspecs
skunert 48267c2
PRDoc one more time
skunert d595c7b
Fix collator image
skunert b1edd47
Merge branch 'master' into slot-based-mvp
skunert 764fcd5
Merge branch 'master' into slot-based-mvp
skunert 62d5b9d
post-merge adjustments
skunert 7bc28ff
Review comments
skunert d986ee7
Apply suggestions from code review
skunert dd36c44
Remove unnecessary pending condition in parent search
skunert 91c6539
Adjust comments and tracing log
skunert 8e0b80d
".git/.scripts/commands/fmt/fmt.sh"
09222f7
make semver happy
skunert 938d6e6
Reviewer comment: Remove duplicate check for pending
skunert File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Use commong async backing params fetching methods
- Loading branch information
commit 8df558f6da6eafa1d9a7755e3260605877c62075
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.