Transfer asset via bridge with pallet xcm by bkontur · Pull Request #2762 · paritytech/cumulus

bkontur · 2023-06-20T09:45:59Z

Relates to: #2528 (BridgeHub for Kusama/Polkadot) - real bridging comes here

PR introduces:

xcm routing configuration for bridging between AssetHubKusama <-> AssetHubPolkadot
- explicitly is allowed only reserve based transfer for KSM/DOT to cross-consensus AssetHubs and nothing else at first (lets start defensively)
  - if we decided, we can allow whatever (ETH, TrustBackedAssets, ForeignAssets, PoolAssets ...)
tests for simulation limited_reserve_transfer_assets with pallet_xcm both ways + handling ReserveAssetDeposisted on both sides
local zomienet run for: (check parachains/runtimes/bridge-hubs/README.md in this PR)
- reserve based transfer of KSMs from AHK to AHP
- reserve based transfer of DOTs from AHP to AHK

TODOs:

set filters for "reserve asset transfer" on both sides
add xcm-emulator tests

* Dumy weights to get compile * Change UniversalLocation according to paritytech/polkadot#4097 (Location Inversion Removed) * Fix bridge-hubs weights * ".git/.scripts/bench-bot.sh" pallet statemine assets pallet_xcm * ".git/.scripts/bench-bot.sh" pallet statemint assets pallet_xcm * ".git/.scripts/bench-bot.sh" pallet collectives-polkadot collectives pallet_xcm * ".git/.scripts/bench-bot.sh" pallet westmint assets pallet_xcm * ".git/.scripts/bench-bot.sh" xcm bridge-hub-kusama bridge-hubs pallet_xcm_benchmarks::generic * ".git/.scripts/bench-bot.sh" xcm bridge-hub-kusama bridge-hubs pallet_xcm_benchmarks::fungible * ".git/.scripts/bench-bot.sh" pallet bridge-hub-kusama bridge-hubs pallet_xcm * ".git/.scripts/bench-bot.sh" pallet bridge-hub-rococo bridge-hubs pallet_xcm * ".git/.scripts/bench-bot.sh" xcm bridge-hub-rococo bridge-hubs pallet_xcm_benchmarks::fungible * ".git/.scripts/bench-bot.sh" xcm bridge-hub-rococo bridge-hubs pallet_xcm_benchmarks::generic * Change NetworkId to Option<NetworkId> Co-authored-by: command-bot <> Co-authored-by: Keith Yeung <kungfukeith11@gmail.com>

* Add UpwardMessageSent event in parachain-system * additional fixes * Message Id * Fix errors from merge * fmt * more fmt * Remove todo * more formatting

…achains (#2060) * Allow explicit unpaid executions from the relay chains for system parachains * Put origin-filtering barriers into WithComputedOrigin * Use ConstU32<8> * Small nits * formatting * cargo fmt

…into bko-transfer-asset-via-bridge

…via-bridge-pallet-xcm

joepetrowski

Looking good to me now. Want to take one more pass over it before approving.

parachains/runtimes/assets/common/src/matching.rs

Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com>

vstam1

LGTM

parachains/runtimes/assets/asset-hub-kusama/tests/tests.rs

Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com>

…via-bridge-pallet-xcm

bkontur · 2023-08-16T13:45:26Z

bot fmt

command-bot · 2023-08-16T13:45:30Z

@bkontur https://gitlab.parity.io/parity/mirrors/cumulus/-/jobs/3400918 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/fmt/fmt.sh". Check out https://gitlab.parity.io/parity/mirrors/cumulus/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 36-2df042da-f065-40c9-af94-7c82c6d07ad7 to cancel this command or bot cancel to cancel all commands in this pull request.

command-bot · 2023-08-16T13:46:13Z

@bkontur Command "$PIPELINE_SCRIPTS_DIR/commands/fmt/fmt.sh" has finished. Result: https://gitlab.parity.io/parity/mirrors/cumulus/-/jobs/3400918 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/cumulus/-/jobs/3400918/artifacts/download.

…via-bridge-pallet-xcm

bkontur and others added 30 commits December 22, 2022 12:52

Added withdraw/deposit to bridge's reserve account

e127426

TODO: temporary hack pipelines because of paritytech/devops#2190

a7dd1fc

Merge remote-tracking branch 'origin/master' into gav-xcm-v3

2f39d75

Merge branch 'master' into gav-xcm-v3

7f3727f

Add event for showing the hash of an UMP sent message (#1228)

e2a3354

* Add UpwardMessageSent event in parachain-system * additional fixes * Message Id * Fix errors from merge * fmt * more fmt * Remove todo * more formatting

Merge branch 'master' into gav-xcm-v3

b7970c3

Fixes

63a18c2

Fixes

f9a0390

Fixes

893bc5c

Fixes

e303fb2

Allow receive bridged Xcm::Trap on Westmint

2ff5c4f

Allow (temporary) Statemine send xcm messages

18ad58b

refactor TrustedBridgedNetworks

fe7817e

Allow (hacky) xcm::transact for remark/remark_with_event

a80781e

Unit-test for Statemine - send_xcm_transact_with_remark_with_event_works

0985898

Fix for BridgedSignedAccountId32AsNative

c3195d2

Allow explicit unpaid executions from the relay chains for system par…

8fe432b

…achains (#2060) * Allow explicit unpaid executions from the relay chains for system parachains * Put origin-filtering barriers into WithComputedOrigin * Use ConstU32<8> * Small nits * formatting * cargo fmt

Merge remote-tracking branch 'origin/master' into gav-xcm-v3

e28ef3c

Allow receiving XCMs from any relay chain plurality

87e0ea1

Fixes

146c742

don't rename pallet in runtime

3353263

don't rename and fix tests

9e211b8

Added allowed_target_location + reanchored assets/destination + tests

31bc1c6

fix benchmark helper

25c2fc5

fix benchmark build

a4a581b

Merge remote-tracking branch 'origin/joe-bridge-hub-westmint-assets' …

7fbfeb3

…into bko-transfer-asset-via-bridge

more benchmark fixes

9cd2708

Merge remote-tracking branch 'origin/joe-bridge-hub-westmint-assets' …

23f9e4d

…into bko-transfer-asset-via-bridge

fix penpal and rococo

81e9703

bkontur added 4 commits July 28, 2023 12:50

Improved location filtering and FilteredNetworkExportTable

5103de8

Merge remote-tracking branch 'origin/master' into bko-transfer-asset-…

5c94232

…via-bridge-pallet-xcm

Strict XcmReserveTransferFilter with ExcludeOnlyForRemoteDestination

bae9f30

Periods. :).

db7a045

bkontur requested review from joepetrowski and vstam1 July 28, 2023 12:12

joepetrowski reviewed Jul 31, 2023

View reviewed changes

parachains/runtimes/assets/common/src/matching.rs Outdated Show resolved Hide resolved

paritytech-ci requested a review from a team July 31, 2023 12:31

Update parachains/runtimes/assets/common/src/matching.rs

ce84eb0

Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com>

vstam1 approved these changes Aug 1, 2023

View reviewed changes

bkontur mentioned this pull request Aug 1, 2023

Dynamic fees for bridges-v1 paritytech/parity-bridges-common#2294

Merged

9 tasks

joepetrowski reviewed Aug 2, 2023

View reviewed changes

parachains/runtimes/assets/asset-hub-kusama/tests/tests.rs Outdated Show resolved Hide resolved

bkontur and others added 3 commits August 2, 2023 11:03

Update parachains/runtimes/assets/asset-hub-kusama/tests/tests.rs

8d2812f

Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com>

Variable rename

43d14a9

Merge remote-tracking branch 'origin/master' into bko-transfer-asset-…

5ce9c0f

…via-bridge-pallet-xcm

paritytech-ci requested a review from a team August 2, 2023 09:25

joepetrowski approved these changes Aug 2, 2023

View reviewed changes

bkontur mentioned this pull request Aug 3, 2023

Transfer asset via bridge #2013

Closed

12 tasks

Merge remote-tracking branch 'origin/master' into bko-transfer-asset-…

01ee625

…via-bridge-pallet-xcm

bkontur mentioned this pull request Aug 11, 2023

Transfer asset via bridge pallet xcm with dynamic fees and back-pressure #2997

Open

6 tasks

Merge remote-tracking branch 'origin/master' into bko-transfer-asset-…

fef0d8d

…via-bridge-pallet-xcm

".git/.scripts/commands/fmt/fmt.sh"

f365dca

bkontur added 2 commits August 16, 2023 21:55

Changed min_balance for pKSM 1_000_000_000 -> 10_000_000_000

3355171

Merge remote-tracking branch 'origin/master' into bko-transfer-asset-…

41fe2db

…via-bridge-pallet-xcm

acatangiu mentioned this pull request Aug 25, 2023

[P-K Bridge] AssetHubPolkadot<>AssetHubKusama asset transfer (via pallet-xcm) paritytech/parity-bridges-common#2431

Closed

4 tasks

acatangiu mentioned this pull request Aug 28, 2023

asset-hubs: transfer assets via bridge using pallet-xcm paritytech/polkadot-sdk#1224

Closed

bkontur mentioned this pull request Sep 1, 2023

[asset-hubs/bridge-hubs] Bridging Polkadot Asset Hub <-> Kusama Asset Hub to enable asset transfer of KSMs/DOTs paritytech/polkadot-sdk#1352

Closed

9 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Transfer asset via bridge with pallet xcm#2762

Transfer asset via bridge with pallet xcm#2762
bkontur wants to merge 412 commits intomasterfrom
bko-transfer-asset-via-bridge-pallet-xcm

bkontur commented Jun 20, 2023 •

edited

Loading

Uh oh!

joepetrowski left a comment

Uh oh!

Uh oh!

vstam1 left a comment

Uh oh!

Uh oh!

bkontur commented Aug 16, 2023

Uh oh!

command-bot bot commented Aug 16, 2023 •

edited

Loading

Uh oh!

command-bot bot commented Aug 16, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

Conversation

bkontur commented Jun 20, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

joepetrowski left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

vstam1 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

bkontur commented Aug 16, 2023

Uh oh!

command-bot bot commented Aug 16, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

command-bot bot commented Aug 16, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

bkontur commented Jun 20, 2023 •

edited

Loading

command-bot bot commented Aug 16, 2023 •

edited

Loading