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.

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

Implements PoV export and local validation #4640

Merged
merged 9 commits into from
Jul 19, 2024
Merged

Implements PoV export and local validation #4640

merged 9 commits into from
Jul 19, 2024

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented May 29, 2024

This pull requests adds a new CLI flag to polkadot-parachains --export-pov-to-path. This CLI flag will instruct the node to export any PoV that it build locally to export to the given folder. Then these PoV files can be validated using the introduced cumulus-pov-validator. The combination of export and validation can be used for debugging parachain validation issues that may happen on the relay chain.

This pull requests adds a new CLI flag to `polkadot-parachains`
`--export-pov-to-path`. This CLI flag will instruct the node to export
any `PoV` that it build locally to export to the given folder. Then
these `PoV` files can be validated using the introduced
`cumulus-pov-validator`. The combination of export and validation can be
used for debugging parachain validation issues that may happen on the
relay chain.
@bkchr bkchr added the T9-cumulus This PR/Issue is related to cumulus. label May 29, 2024
async move {
let timestamp = sp_timestamp::InherentDataProvider::from_system_time();

let slot_duration = cumulus_client_consensus_aura::slot_duration(&*client)?;
Copy link
Member

Choose a reason for hiding this comment

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

this change seems unrelated, what are the implications? added support for dynamic slot durations?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes

cumulus/bin/pov-validator/src/main.rs Show resolved Hide resolved
let duration = start.elapsed();

match res {
Ok(_) => tracing::info!("Validation was successful"),
Copy link
Contributor

@sandreim sandreim May 31, 2024

Choose a reason for hiding this comment

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

I would also add an option to decode the commitments as ValidationResult to a file for debugging purposes.

@xlc xlc mentioned this pull request Jul 8, 2024
@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/6739374

@bkchr bkchr enabled auto-merge July 18, 2024 14:52
@bkchr bkchr added this pull request to the merge queue Jul 19, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 19, 2024
@bkchr bkchr added this pull request to the merge queue Jul 19, 2024
Merged via the queue into master with commit d649746 Jul 19, 2024
157 of 160 checks passed
@bkchr bkchr deleted the bkchr-export-pov branch July 19, 2024 22:03
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
This pull requests adds a new CLI flag to `polkadot-parachains`
`--export-pov-to-path`. This CLI flag will instruct the node to export
any `PoV` that it build locally to export to the given folder. Then
these `PoV` files can be validated using the introduced
`cumulus-pov-validator`. The combination of export and validation can be
used for debugging parachain validation issues that may happen on the
relay chain.
@skunert skunert mentioned this pull request Aug 7, 2024
2 tasks
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
None yet
Development

Successfully merging this pull request may close these issues.

4 participants