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

Remove stale extension version from UncheckedExtrinsic v4 structure #6404

Closed
wants to merge 8 commits into from

Conversation

georgepisaltu
Copy link
Contributor

Follow up to #3685

The main PR introduced bare support for the new extension version byte. However, the extension version byte is not used for the old signed extrinsic type in version 4. This PR removes the byte from that structure, previously unused and defaulted to 0.

In this same PR, new weights will be generated for the modules affected by the main PR.

@georgepisaltu georgepisaltu added the T1-FRAME This PR/Issue is related to core FRAME, the framework. label Nov 7, 2024
@georgepisaltu georgepisaltu self-assigned this Nov 7, 2024
@georgepisaltu georgepisaltu requested a review from a team as a code owner November 7, 2024 13:12
@georgepisaltu
Copy link
Contributor Author

bot bench-all substrate -v PIPELINE_SCRIPTS_REF=george/substrate-ext

@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719043 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --target_dir=substrate --features=riscv. Check out https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 48-ebb9c2fd-8f26-45c1-9d8c-c1f961652d92 to cancel this command or bot cancel to cancel all commands in this pull request.

@georgepisaltu
Copy link
Contributor Author

bot bench-all cumulus -v PIPELINE_SCRIPTS_REF=george/substrate-ext

@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719044 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --runtime=asset-hub-rococo --target_dir=cumulus. Check out https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 49-aca594ff-91ac-4497-b5f9-6ed2d8013c7e to cancel this command or bot cancel to cancel all commands in this pull request.

@georgepisaltu
Copy link
Contributor Author

bot bench-all polkadot -v PIPELINE_SCRIPTS_REF=george/substrate-ext

@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719045 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --runtime=rococo --target_dir=polkadot. Check out https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 50-0a7461ac-771d-414f-b784-ab48badd473d to cancel this command or bot cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu Command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --target_dir=substrate --features=riscv has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719043 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719043/artifacts/download.

@georgepisaltu
Copy link
Contributor Author

bot bench-all substrate -v PIPELINE_SCRIPTS_REF=george/substrate-ext

@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719176 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --target_dir=substrate --features=riscv. Check out https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 51-10ec249c-ad9a-4d71-9f28-51d7d7e2e731 to cancel this command or bot cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu Command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --target_dir=substrate --features=riscv has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719176 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719176/artifacts/download.

@georgepisaltu
Copy link
Contributor Author

bot bench-all substrate -v PIPELINE_SCRIPTS_REF=george/substrate-ext

@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719627 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --target_dir=substrate --features=riscv. Check out https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 52-7083c6b1-1d15-4867-8354-075c45410048 to cancel this command or bot cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu Command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --target_dir=substrate --features=riscv has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719627 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719627/artifacts/download.

@georgepisaltu
Copy link
Contributor Author

bot bench-all substrate -v PIPELINE_SCRIPTS_REF=george/substrate-ext

@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719693 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --target_dir=substrate. Check out https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 53-c537172e-25e5-480f-8b7e-cf658136abe9 to cancel this command or bot cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu Command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --runtime=asset-hub-rococo --target_dir=cumulus has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719044 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719044/artifacts/download.

lexnv and others added 3 commits November 7, 2024 19:36
…6380)

This PR ensures that external addresses with different PeerIDs are not
propagated to the higher layer of the network code.

While at it, this ensures that libp2p only adds the `/p2p/peerid` part
to the discovered address if it does not contain it already.

This is a followup from:
- #6298

cc @paritytech/networking

---------

Signed-off-by: Alexandru Vasile <[email protected]>
Co-authored-by: Dmitry Markin <[email protected]>
Fixes #5530

This PR introduces the removal of backing jobs that have been back
pressured for longer than `allowedAncestryLen`, as these candidates are
no longer viable.

It is reasonable to expect a result for a backing job execution within
`allowedAncestryLen` blocks. Therefore, we set the job TTL as a relay
block number and synchronize the validation host by sending activated
leaves.

---------

Co-authored-by: Andrei Sandu <[email protected]>
Co-authored-by: Branislav Kontur <[email protected]>
@command-bot
Copy link

command-bot bot commented Nov 7, 2024

@georgepisaltu Command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --target_dir=substrate has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719693 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719693/artifacts/download.

@@ -632,6 +632,7 @@ async fn request_candidate_validation(
) -> Result<ValidationResult, Error> {
let (tx, rx) = oneshot::channel();
let is_system = candidate_receipt.descriptor.para_id().is_system();
let relay_parent = candidate_receipt.descriptor.relay_parent();
Copy link
Contributor

Choose a reason for hiding this comment

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

why all these changes in backing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The bot rebased or formatted the code. You can see my initial commit here, the rest of the changes are pushed by the bot.

@command-bot
Copy link

command-bot bot commented Nov 8, 2024

@georgepisaltu Command "$PIPELINE_SCRIPTS_DIR/commands/bench-all/bench-all.sh" --runtime=rococo --target_dir=polkadot has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719045 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7719045/artifacts/download.

@georgepisaltu georgepisaltu changed the title Remove stale extenstion version from UncheckedExtrinsic v4 structure Remove stale extension version from UncheckedExtrinsic v4 structure Nov 8, 2024
Signed-off-by: georgepisaltu <[email protected]>
@georgepisaltu
Copy link
Contributor Author

Closing in favor of #6418

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T1-FRAME This PR/Issue is related to core FRAME, the framework.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants