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

Setup system parachain runtimes benchmarking for delivering to sibling parachains instead of Parent #546

Open
4 tasks
bkontur opened this issue Jan 16, 2025 · 2 comments

Comments

@bkontur
Copy link
Contributor

bkontur commented Jan 16, 2025

Description

Currently, all system parachains benchmarking setups for pallet_xcm::benchmarking::Config or pallet_xcm_benchmarks::Config are written to deliver to the Parent (i.e., the relay chain), such as for all asset transfers.

As we are transitioning functionality, like balances, from the relay chain to the asset hubs, we need to update these benchmarks to reflect this migration.

Relevant Changes

Delivery Helpers:
Replace DeliveryHelpers that use cumulus_primitives_utility::ToParentDeliveryHelper (example) with polkadot_runtime_common::xcm_sender::ToParachainDeliveryHelper (example).

Destination Updates:
Review and update all destinations used in the benchmarks, including:

  • fn valid_destination()
  • fn subscribe_origin()
  • fn set_up_complex_asset_transfer()
  • TrustedTeleporter
  • ...

This ensures the benchmarks are aligned with the changes in architecture and accurately reflect the new delivery paths.

TODO/Progress

  • revisit/fix all system parachain benchmark setup for pallet_xcm::benchmarking::Config
  • revisit/fix all system parachain benchmark setup for pallet_xcm_benchmarks::Config
  • check all other non-xcm benchmarks
  • investigate coretime (uses calls to Parent) ?
@Nathy-bajo
Copy link

@bkontur I want to try attempting this!

@bkontur
Copy link
Contributor Author

bkontur commented Jan 22, 2025

@bkontur I want to try attempting this!

Actually, there’s an equivalent issue for testnets, which I would prefer to have completed first. This allows us to catch, test, and refine things there. Later, we can simply replicate the relevant changes to the Fellows runtimes.

However, that issue is already claimed. That said, you’re more than welcome to participate in paritytech/polkadot-sdk#7211, at least as a reviewer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants