Bridges lane id agnostic for backwards compatibility (#5649)
This PR primarily fixes the issue with `zombienet-bridges-0001-asset-transfer-works` (see: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7404903). The PR looks large, but most of the changes involve splitting `LaneId` into `LegacyLaneId` and `HashedLaneId`. All pallets now use `LaneId` as a generic parameter. The actual bridging pallets are now backward compatible and work with actual **substrate-relay v1.6.10**, which does not even known anything about permissionless lanes or the new pallet changes. ## Important - [x] added migration for `pallet_bridge_relayers` and `RewardsAccountParams` change order of params, which generates different accounts ## Deployment follow ups - [ ] fix monitoring for `at_{}_relay_{}_reward_for_msgs_from_{}_on_lane_{}` - [ ] check sovereign reward accounts - because of changed `RewardsAccountParams` - [ ] deploy another messages instances for permissionless lanes - on BHs or AHs? - [ ] return back `open_and_close_bridge_works` for another `pallet-bridge-messages` instance --------- Signed-off-by:Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io> Co-authored-by:
Adrian Catangiu <adrian@parity.io> Co-authored-by:
Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Showing
- Cargo.lock 7 additions, 0 deletionsCargo.lock
- bridges/bin/runtime-common/src/extensions.rs 3 additions, 8 deletionsbridges/bin/runtime-common/src/extensions.rs
- bridges/bin/runtime-common/src/messages_api.rs 3 additions, 5 deletionsbridges/bin/runtime-common/src/messages_api.rs
- bridges/bin/runtime-common/src/messages_benchmarking.rs 43 additions, 35 deletionsbridges/bin/runtime-common/src/messages_benchmarking.rs
- bridges/bin/runtime-common/src/mock.rs 17 additions, 12 deletionsbridges/bin/runtime-common/src/mock.rs
- bridges/chains/chain-bridge-hub-kusama/src/lib.rs 1 addition, 1 deletionbridges/chains/chain-bridge-hub-kusama/src/lib.rs
- bridges/chains/chain-bridge-hub-polkadot/src/lib.rs 1 addition, 1 deletionbridges/chains/chain-bridge-hub-polkadot/src/lib.rs
- bridges/chains/chain-bridge-hub-rococo/src/lib.rs 1 addition, 1 deletionbridges/chains/chain-bridge-hub-rococo/src/lib.rs
- bridges/chains/chain-bridge-hub-westend/src/lib.rs 1 addition, 1 deletionbridges/chains/chain-bridge-hub-westend/src/lib.rs
- bridges/chains/chain-polkadot-bulletin/src/lib.rs 1 addition, 1 deletionbridges/chains/chain-polkadot-bulletin/src/lib.rs
- bridges/modules/messages/src/benchmarking.rs 9 additions, 9 deletionsbridges/modules/messages/src/benchmarking.rs
- bridges/modules/messages/src/call_ext.rs 15 additions, 15 deletionsbridges/modules/messages/src/call_ext.rs
- bridges/modules/messages/src/inbound_lane.rs 6 additions, 4 deletionsbridges/modules/messages/src/inbound_lane.rs
- bridges/modules/messages/src/lanes_manager.rs 17 additions, 15 deletionsbridges/modules/messages/src/lanes_manager.rs
- bridges/modules/messages/src/lib.rs 52 additions, 30 deletionsbridges/modules/messages/src/lib.rs
- bridges/modules/messages/src/outbound_lane.rs 4 additions, 2 deletionsbridges/modules/messages/src/outbound_lane.rs
- bridges/modules/messages/src/proofs.rs 22 additions, 21 deletionsbridges/modules/messages/src/proofs.rs
- bridges/modules/messages/src/tests/messages_generation.rs 12 additions, 4 deletionsbridges/modules/messages/src/tests/messages_generation.rs
- bridges/modules/messages/src/tests/mock.rs 50 additions, 45 deletionsbridges/modules/messages/src/tests/mock.rs
- bridges/modules/messages/src/tests/pallet_tests.rs 14 additions, 11 deletionsbridges/modules/messages/src/tests/pallet_tests.rs