Skip to content

Omninode instant seal: Support relay parent offset#10658

Merged
bkchr merged 3 commits intomasterfrom
bkchr-omni-relay-parent-descendants
Dec 16, 2025
Merged

Omninode instant seal: Support relay parent offset#10658
bkchr merged 3 commits intomasterfrom
bkchr-omni-relay-parent-descendants

Conversation

@bkchr
Copy link
Copy Markdown
Member

@bkchr bkchr commented Dec 16, 2025

This brings support for relay parent offset to omni-node instant seal.

Closes: #9886

This brings support for relay parent offset to omni-node instant seal.
@bkchr bkchr requested review from lexnv and skunert December 16, 2025 08:03
@bkchr bkchr added the T9-cumulus This PR/Issue is related to cumulus. label Dec 16, 2025
@bkchr
Copy link
Copy Markdown
Member Author

bkchr commented Dec 16, 2025

/cmd prdoc --audience node_dev --bum patch

@bkchr
Copy link
Copy Markdown
Member Author

bkchr commented Dec 16, 2025

/cmd prdoc --audience node_dev --bump major

@github-actions
Copy link
Copy Markdown
Contributor

Command "prdoc --audience node_dev --bump major" has failed ❌! See logs here

sproof_builder.included_para_head = self.current_para_block_head.clone();
sproof_builder.num_authorities = 2;
let (relay_parent_storage_root, proof, relay_parent_descendants) =
sproof_builder.into_state_root_proof_and_descendants(1);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm how did this work with a value of 1 before, did we break it in previous PR?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was just introduced last week and probably never tested :P

@bkchr bkchr added this pull request to the merge queue Dec 16, 2025
Merged via the queue into master with commit 5165ea2 Dec 16, 2025
226 of 230 checks passed
@bkchr bkchr deleted the bkchr-omni-relay-parent-descendants branch December 16, 2025 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T9-cumulus This PR/Issue is related to cumulus.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Omni-node: Support relay parent offset in manual-seal mode

4 participants