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.

Sign up for GitHub

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

Jump to bottom

lookahead collator: only build on top of scheduled relay parents #1429

Merged
merged 4 commits into from
Sep 12, 2023
Merged

lookahead collator: only build on top of scheduled relay parents #1429

merged 4 commits into from
Sep 12, 2023

Conversation

slumber
Copy link
Contributor

@slumber slumber commented Sep 6, 2023 •
edited
Loading

Partially addresses #1400

@slumber
lookahead collator: filter out relay parents
23698f7 
if the parachain is not scheduled
@slumber slumber added the T9-cumulus This PR/Issue is related to cumulus. label Sep 6, 2023
@slumber slumber requested review from rphmeier and sandreim September 6, 2023 14:47
@sandreim
Copy link
Contributor

sandreim commented Sep 7, 2023

With async backing enabled, no candidates are backed with this PR and latest Westend(master) runtime.

@sandreim
Copy link
Contributor

sandreim commented Sep 7, 2023

With async backing enabled, no candidates are backed with this PR and latest Westend(master) runtime.

Actually I haven't built it with network-protocol-staging and that explains the issue.

@paritytech-ci paritytech-ci requested review from a team September 7, 2023 12:08
@sandreim
Copy link
Contributor

sandreim commented Sep 7, 2023

Looks good on Versi!

@rphmeier
Copy link
Contributor

rphmeier commented Sep 8, 2023

The lookahead collator should only build on chains where all in-scope and off-chain blocks are built on allowed relay-parents.

This PR only does half of the issue.
It changes collators not to build on disallowed relay-parents, but collators will still build on top of blocks which had disallowed relay parents. The parent search would need to be altered in order to fix this.

I'm OK with merging as-is, since an entirely-honest collator set would have no issues. But we should leave #1400 open

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3659191

@slumber slumber force-pushed the slumber-lookahead-on-scheduled-rp branch from 98121a5 to 345d613 Compare September 12, 2023 13:33
@slumber slumber enabled auto-merge (squash) September 12, 2023 13:37
@slumber slumber merged commit 09630fc into master Sep 12, 2023
@slumber slumber deleted the slumber-lookahead-on-scheduled-rp branch September 12, 2023 14:15
@kiltbot kiltbot mentioned this pull request Oct 19, 2023
Closed
@ahmadkaouk ahmadkaouk mentioned this pull request Nov 16, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sandreim sandreim sandreim approved these changes

@rphmeier rphmeier rphmeier approved these changes

Assignees
No one assigned
Labels
T9-cumulus This PR/Issue is related to cumulus.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@slumber @sandreim @rphmeier @paritytech-cicd-pr