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

Switch runtimes benchmarking for delivering to sibling parachains instead of Parent #7321

Open
wants to merge 38 commits into
base: master
Choose a base branch
from

Conversation

EleisonC
Copy link
Contributor

Title: Update XCM benchmarks for sibling parachain delivery (closes #7211)

Description:

This PR updates XCM benchmarking configurations for testnet system parachains to reflect delivery to sibling parachains instead of the Parent relay chain.

Integration:

Replaced ToParentDeliveryHelper with ToParachainDeliveryHelper.

Updated benchmark destinations

@EleisonC EleisonC changed the title switch destination Parent to AH Switch runtimes benchmarking for delivering to sibling parachains instead of Parent Jan 23, 2025
@EleisonC
Copy link
Contributor Author

Hey @bkontur

I am Kindly requesting for your feedback on this. PR
runtime - assets hub westend

Copy link
Contributor

@bkontur bkontur left a comment

Choose a reason for hiding this comment

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

the point here for AssetHubWestend was to change location Parent to the RandomParaLocation for delivery (but only those Parent which represents destination and not the AssetId)

EleisonC and others added 17 commits January 27, 2025 19:57
@EleisonC
Copy link
Contributor Author

EleisonC commented Jan 27, 2025

the point here for AssetHubWestend was to change location Parent to the RandomParaLocation for delivery (but only those Parent which represents destination and not the AssetId)

Hey @bkontur
I was able to review the changes you made. They provide more clarity. Let me get to the others
Something like
Screenshot 2025-01-27 at 22 37 01

@EleisonC
Copy link
Contributor Author

Hey @bkontur, I have refactored the locations in the asset-hub-rococo. Could you kindly let me know if we are on track?

@EleisonC EleisonC requested a review from bkontur February 1, 2025 04:10
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

Successfully merging this pull request may close these issues.

Setup testnet system parachain runtimes benchmarking for delivering to sibling parachains instead of Parent
2 participants