From 9ec8009c5a8fdf89499fcd2a40df0292d3950efa Mon Sep 17 00:00:00 2001 From: Branislav Kontur <bkontur@gmail.com> Date: Thu, 28 Nov 2024 12:41:09 +0100 Subject: [PATCH] Multiple instances for pallet-bridge-relayers fix (#6684) Previously, we added multi-instance pallet support for `pallet-bridge-relayers`, but missed fixing it in this one place. --------- Co-authored-by: GitHub Action <action@github.com> Co-authored-by: Adrian Catangiu <adrian@parity.io> --- bridges/modules/relayers/src/extension/mod.rs | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/bridges/modules/relayers/src/extension/mod.rs b/bridges/modules/relayers/src/extension/mod.rs index 34d280d26d6..d562ed9bcd0 100644 --- a/bridges/modules/relayers/src/extension/mod.rs +++ b/bridges/modules/relayers/src/extension/mod.rs @@ -129,7 +129,7 @@ pub struct BridgeRelayersTransactionExtension<Runtime, Config, LaneId>( impl<R, C, LaneId> BridgeRelayersTransactionExtension<R, C, LaneId> where Self: 'static + Send + Sync, - R: RelayersConfig<LaneId = LaneId> + R: RelayersConfig<C::BridgeRelayersPalletInstance, LaneId = LaneId> + BridgeMessagesConfig<C::BridgeMessagesPalletInstance, LaneId = LaneId> + TransactionPaymentConfig, C: ExtensionConfig<Runtime = R, LaneId = LaneId>, @@ -250,7 +250,7 @@ where // let's also replace the weight of slashing relayer with the weight of rewarding relayer if call_info.is_receive_messages_proof_call() { post_info_weight = post_info_weight.saturating_sub( - <R as RelayersConfig>::WeightInfo::extra_weight_of_successful_receive_messages_proof_call(), + <R as RelayersConfig<C::BridgeRelayersPalletInstance>>::WeightInfo::extra_weight_of_successful_receive_messages_proof_call(), ); } @@ -278,7 +278,7 @@ impl<R, C, LaneId> TransactionExtension<R::RuntimeCall> for BridgeRelayersTransactionExtension<R, C, LaneId> where Self: 'static + Send + Sync, - R: RelayersConfig<LaneId = LaneId> + R: RelayersConfig<C::BridgeRelayersPalletInstance, LaneId = LaneId> + BridgeMessagesConfig<C::BridgeMessagesPalletInstance, LaneId = LaneId> + TransactionPaymentConfig, C: ExtensionConfig<Runtime = R, LaneId = LaneId>, @@ -326,7 +326,9 @@ where }; // we only boost priority if relayer has staked required balance - if !RelayersPallet::<R>::is_registration_active(&data.relayer) { + if !RelayersPallet::<R, C::BridgeRelayersPalletInstance>::is_registration_active( + &data.relayer, + ) { return Ok((Default::default(), Some(data), origin)) } @@ -382,7 +384,11 @@ where match call_result { RelayerAccountAction::None => (), RelayerAccountAction::Reward(relayer, reward_account, reward) => { - RelayersPallet::<R>::register_relayer_reward(reward_account, &relayer, reward); + RelayersPallet::<R, C::BridgeRelayersPalletInstance>::register_relayer_reward( + reward_account, + &relayer, + reward, + ); log::trace!( target: LOG_TARGET, @@ -394,7 +400,7 @@ where ); }, RelayerAccountAction::Slash(relayer, slash_account) => - RelayersPallet::<R>::slash_and_deregister( + RelayersPallet::<R, C::BridgeRelayersPalletInstance>::slash_and_deregister( &relayer, ExplicitOrAccountParams::Params(slash_account), ), -- GitLab