git.delta.rocks / unique-network / refs/commits / 540966542a36

difftreelog

feat(configuration) benchmarks

Fahrrader2022-12-28parent: #63282b0.patch.diff
in: master

15 files changed

modifiedCargo.lockdiffbeforeafterboth
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -5846,6 +5846,7 @@
 version = "0.1.2"
 dependencies = [
  "fp-evm",
+ "frame-benchmarking",
  "frame-support",
  "frame-system",
  "parity-scale-codec 3.2.1",
modifiedMakefilediffbeforeafterboth
--- a/Makefile
+++ b/Makefile
@@ -89,6 +89,10 @@
 bench-evm-migration:
 	make _bench PALLET=evm-migration
 
+.PHONY: bench-configuration
+bench-configuration:
+	make _bench PALLET=configuration
+
 .PHONY: bench-common
 bench-common:
 	make _bench PALLET=common
@@ -143,4 +147,4 @@
 	
 .PHONY: bench
 # Disabled: bench-scheduler, bench-rmrk-core, bench-rmrk-equip
-bench: bench-evm-migration bench-unique bench-structure bench-fungible bench-refungible bench-nonfungible bench-foreign-assets bench-collator-selection bench-identity
+bench: bench-evm-migration bench-unique bench-structure bench-fungible bench-refungible bench-nonfungible bench-configuration bench-foreign-assets bench-collator-selection bench-identity
modifiedpallets/collator-selection/src/benchmarking.rsdiffbeforeafterboth
--- a/pallets/collator-selection/src/benchmarking.rs
+++ b/pallets/collator-selection/src/benchmarking.rs
@@ -52,9 +52,6 @@
 };
 use sp_std::prelude::*;
 
-/*pub type BalanceOf<T> =
-<<T as Config>::Currency as Currency<<T as frame_system::Config>::AccountId>>::Balance;*/
-
 const SEED: u32 = 0;
 
 // TODO: remove if this is given in substrate commit.
@@ -116,14 +113,24 @@
 	validators.into_iter().map(|(who, _)| who).collect()
 }
 
+fn register_invulnerables<T: Config + configuration::Config>(count: u32) {
+	let candidates = (0..count)
+		.map(|c| account("candidate", c, SEED))
+		.collect::<Vec<_>>();
+
+	for who in candidates {
+		<CollatorSelection<T>>::add_invulnerable(T::UpdateOrigin::successful_origin(), who).unwrap();
+	}
+}
+
 fn register_candidates<T: Config + configuration::Config>(count: u32) {
 	let candidates = (0..count)
 		.map(|c| account("candidate", c, SEED))
 		.collect::<Vec<_>>();
-	assert!(
+	/*assert!(
 		<LicenseBond<T>>::get() > 0u32.into(),
 		"Bond cannot be zero!"
-	);
+	);*/
 
 	for who in candidates {
 		T::Currency::make_free_balance_be(&who, <LicenseBond<T>>::get() * 2u32.into());
@@ -132,16 +139,45 @@
 	}
 }
 
+fn get_licenses<T: Config + configuration::Config>(count: u32) {
+	let candidates = (0..count)
+		.map(|c| account("candidate", c, SEED))
+		.collect::<Vec<_>>();
+	/*assert!(
+		<LicenseBond<T>>::get() > 0u32.into(),
+		"Bond cannot be zero!"
+	);*/
+
+	for who in candidates {
+		T::Currency::make_free_balance_be(&who, <LicenseBond<T>>::get() * 2u32.into());
+		<CollatorSelection<T>>::get_license(RawOrigin::Signed(who.clone()).into()).unwrap();
+	}
+}
+
 benchmarks! {
 	where_clause { where T: pallet_authorship::Config + session::Config + configuration::Config }
 
 	add_invulnerable {
-		let b in 1 .. T::MaxCollators::get();
-		let new_invulnerable = register_validators::<T>(b)[0].clone();
-		let origin = T::UpdateOrigin::successful_origin();
+		let b in 1 .. T::MaxCollators::get() - 3;
+		register_validators::<T>(b);
+		register_invulnerables::<T>(b);
+
+		// log::info!("{} {}", <Invulnerables<T>>::get().len(), b);
+
+		let new_invulnerable: T::AccountId = whitelisted_caller();
+		let bond: BalanceOf<T> = T::Currency::minimum_balance() * 2u32.into();
+		T::Currency::make_free_balance_be(&new_invulnerable, bond.clone());
+
+		<session::Pallet<T>>::set_keys(
+			RawOrigin::Signed(new_invulnerable.clone()).into(),
+			keys::<T>(b + 1),
+			Vec::new()
+		).unwrap();
+
+		let root_origin = T::UpdateOrigin::successful_origin();
 	}: {
 		assert_ok!(
-			<CollatorSelection<T>>::add_invulnerable(origin, new_invulnerable.clone())
+			<CollatorSelection<T>>::add_invulnerable(root_origin, new_invulnerable.clone())
 		);
 	}
 	verify {
@@ -150,52 +186,28 @@
 
 	remove_invulnerable {
 		let b in 1 .. T::MaxCollators::get();
-		let new_invulnerable = register_validators::<T>(b)[0].clone();
-		let origin = T::UpdateOrigin::successful_origin();
-		assert_ok!(
-			<CollatorSelection<T>>::add_invulnerable(origin.clone(), new_invulnerable.clone())
-		);
-	}: {
-		assert_ok!(
-			<CollatorSelection<T>>::remove_invulnerable(origin, new_invulnerable.clone())
-		);
-	}
-	verify {
-		assert_last_event::<T>(Event::InvulnerableRemoved{invulnerable: new_invulnerable}.into());
-	}
+		register_validators::<T>(b);
+		register_invulnerables::<T>(b);
 
-	/*set_desired_collators {
-		let max: u32 = 999;
-		let origin = T::UpdateOrigin::successful_origin();
+		let root_origin = T::UpdateOrigin::successful_origin();
+		let leaving = <Invulnerables<T>>::get().last().unwrap().clone();
+		whitelist!(leaving);
 	}: {
 		assert_ok!(
-			<CollatorSelection<T>>::set_desired_collators(origin, max.clone())
+			<CollatorSelection<T>>::remove_invulnerable(root_origin, leaving.clone())
 		);
 	}
 	verify {
-		assert_last_event::<T>(Event::NewDesiredCollators{desired_collators: max}.into());
+		assert_last_event::<T>(Event::InvulnerableRemoved{invulnerable: leaving}.into());
 	}
 
-	set_license_bond {
-		let bond_amount: BalanceOf<T> = T::Currency::minimum_balance() * 10u32.into();
-		let origin = T::UpdateOrigin::successful_origin();
-	}: {
-		assert_ok!(
-			<CollatorSelection<T>>::set_license_bond(origin, bond_amount.clone())
-		);
-	}
-	verify {
-		assert_last_event::<T>(Event::NewLicenseBond{bond_amount}.into());
-	}*/
-
 	get_license {
 		let c in 1 .. T::MaxCollators::get();
 
 		<LicenseBond<T>>::put(T::Currency::minimum_balance());
-		<DesiredCollators<T>>::put(c + 1);
 
 		register_validators::<T>(c);
-		register_candidates::<T>(c);
+		get_licenses::<T>(c);
 
 		let caller: T::AccountId = whitelisted_caller();
 		let bond: BalanceOf<T> = T::Currency::minimum_balance() * 2u32.into();
@@ -215,10 +227,10 @@
 	// worst case is when we have all the max-candidate slots filled except one, and we fill that
 	// one.
 	onboard {
-		let c in 1 .. T::MaxCollators::get();
+		let c in 1 .. 5;
 
 		<LicenseBond<T>>::put(T::Currency::minimum_balance());
-		<DesiredCollators<T>>::put(c + 1);
+		<DesiredCollators<T>>::put(c + 2);
 
 		register_validators::<T>(c);
 		register_candidates::<T>(c);
@@ -247,7 +259,7 @@
 	offboard {
 		let c in 1 .. T::MaxCollators::get();
 		<LicenseBond<T>>::put(T::Currency::minimum_balance());
-		<DesiredCollators<T>>::put(c);
+		<DesiredCollators<T>>::put(c + 2);
 
 		register_validators::<T>(c);
 		register_candidates::<T>(c);
modifiedpallets/collator-selection/src/lib.rsdiffbeforeafterboth
285	impl<T: Config> Pallet<T> {285	impl<T: Config> Pallet<T> {
286		/// Add a collator to the list of invulnerable (fixed) collators.286		/// Add a collator to the list of invulnerable (fixed) collators.
287		#[pallet::call_index(0)]287		#[pallet::call_index(0)]
288		#[pallet::weight(T::WeightInfo::add_invulnerable(T::MaxCollators::get()))]288		#[pallet::weight(<T as Config>::WeightInfo::add_invulnerable(T::MaxCollators::get()))]
289		pub fn add_invulnerable(289		pub fn add_invulnerable(
290			origin: OriginFor<T>,290			origin: OriginFor<T>,
291			new: T::AccountId,291			new: T::AccountId,
315315
316		/// Remove a collator from the list of invulnerable (fixed) collators.316		/// Remove a collator from the list of invulnerable (fixed) collators.
317		#[pallet::call_index(1)]317		#[pallet::call_index(1)]
318		#[pallet::weight(T::WeightInfo::remove_invulnerable(T::MaxCollators::get()))]318		#[pallet::weight(<T as Config>::WeightInfo::remove_invulnerable(T::MaxCollators::get()))]
319		pub fn remove_invulnerable(319		pub fn remove_invulnerable(
320			origin: OriginFor<T>,320			origin: OriginFor<T>,
321			who: T::AccountId,321			who: T::AccountId,
344		///344		///
345		/// This call is not available to `Invulnerable` collators.345		/// This call is not available to `Invulnerable` collators.
346		#[pallet::call_index(2)]346		#[pallet::call_index(2)]
347		#[pallet::weight(T::WeightInfo::get_license(T::MaxCollators::get()))]347		#[pallet::weight(<T as Config>::WeightInfo::get_license(T::MaxCollators::get()))]
348		pub fn get_license(origin: OriginFor<T>) -> DispatchResultWithPostInfo {348		pub fn get_license(origin: OriginFor<T>) -> DispatchResultWithPostInfo {
349			// register_as_candidate349			// register_as_candidate
350			let who = ensure_signed(origin)?;350			let who = ensure_signed(origin)?;
377		///377		///
378		/// This call is not available to `Invulnerable` collators.378		/// This call is not available to `Invulnerable` collators.
379		#[pallet::call_index(3)]379		#[pallet::call_index(3)]
380		#[pallet::weight(T::WeightInfo::onboard(T::MaxCollators::get()))]380		#[pallet::weight(<T as Config>::WeightInfo::onboard(T::MaxCollators::get()))]
381		pub fn onboard(origin: OriginFor<T>) -> DispatchResultWithPostInfo {381		pub fn onboard(origin: OriginFor<T>) -> DispatchResultWithPostInfo {
382			// register_as_candidate382			// register_as_candidate
383			let who = ensure_signed(origin)?;383			let who = ensure_signed(origin)?;
417				})?;417				})?;
418418
419			Self::deposit_event(Event::CandidateAdded { account_id: who });419			Self::deposit_event(Event::CandidateAdded { account_id: who });
420			Ok(Some(T::WeightInfo::onboard(current_count as u32)).into())420			Ok(Some(<T as Config>::WeightInfo::onboard(current_count as u32)).into())
421		}421		}
422422
423		/// Deregister `origin` as a collator candidate. Note that the collator can only leave on423		/// Deregister `origin` as a collator candidate. Note that the collator can only leave on
424		/// session change. The license to `onboard` later at any other time will remain.424		/// session change. The license to `onboard` later at any other time will remain.
425		#[pallet::call_index(4)]425		#[pallet::call_index(4)]
426		#[pallet::weight(T::WeightInfo::offboard(T::MaxCollators::get()))]426		#[pallet::weight(<T as Config>::WeightInfo::offboard(T::MaxCollators::get()))]
427		pub fn offboard(origin: OriginFor<T>) -> DispatchResultWithPostInfo {427		pub fn offboard(origin: OriginFor<T>) -> DispatchResultWithPostInfo {
428			// leave_intent428			// leave_intent
429			let who = ensure_signed(origin)?;429			let who = ensure_signed(origin)?;
430			let current_count = Self::try_remove_candidate(&who)?;430			let current_count = Self::try_remove_candidate(&who)?;
431431
432			Ok(Some(T::WeightInfo::offboard(current_count as u32)).into())432			Ok(Some(<T as Config>::WeightInfo::offboard(current_count as u32)).into())
433		}433		}
434434
435		/// Forfeit `origin`'s own license. The `LicenseBond` will be unreserved immediately.435		/// Forfeit `origin`'s own license. The `LicenseBond` will be unreserved immediately.
436		///436		///
437		/// This call is not available to `Invulnerable` collators.437		/// This call is not available to `Invulnerable` collators.
438		#[pallet::call_index(5)]438		#[pallet::call_index(5)]
439		#[pallet::weight(T::WeightInfo::release_license(T::MaxCollators::get()))]439		#[pallet::weight(<T as Config>::WeightInfo::release_license(T::MaxCollators::get()))]
440		pub fn release_license(origin: OriginFor<T>) -> DispatchResultWithPostInfo {440		pub fn release_license(origin: OriginFor<T>) -> DispatchResultWithPostInfo {
441			// leave_intent441			// leave_intent
442			let who = ensure_signed(origin)?;442			let who = ensure_signed(origin)?;
443443
444			let current_count = Self::try_remove_candidate_and_release_license(&who, false, true)?;444			let current_count = Self::try_remove_candidate_and_release_license(&who, false, true)?;
445445
446			Ok(Some(T::WeightInfo::release_license(current_count as u32)).into())446			Ok(Some(<T as Config>::WeightInfo::release_license(
447				current_count as u32,
448			))
449			.into())
452		///455		///
453		/// This call is, of course, not applicable to `Invulnerable` collators.456		/// This call is, of course, not applicable to `Invulnerable` collators.
454		#[pallet::call_index(6)]457		#[pallet::call_index(6)]
455		#[pallet::weight(T::WeightInfo::force_release_license(T::MaxCollators::get()))]458		#[pallet::weight(<T as Config>::WeightInfo::force_release_license(T::MaxCollators::get()))]
456		pub fn force_release_license(459		pub fn force_release_license(
457			origin: OriginFor<T>,460			origin: OriginFor<T>,
458			who: T::AccountId,461			who: T::AccountId,
462465
463			let current_count = Self::try_remove_candidate_and_release_license(&who, false, true)?;466			let current_count = Self::try_remove_candidate_and_release_license(&who, false, true)?;
464467
465			Ok(Some(T::WeightInfo::force_release_license(current_count as u32)).into())468			Ok(Some(<T as Config>::WeightInfo::force_release_license(
469				current_count as u32,
470			))
471			.into())
599			<LastAuthoredBlock<T>>::insert(author, frame_system::Pallet::<T>::block_number());605			<LastAuthoredBlock<T>>::insert(author, frame_system::Pallet::<T>::block_number());
600606
601			frame_system::Pallet::<T>::register_extra_weight_unchecked(607			frame_system::Pallet::<T>::register_extra_weight_unchecked(
602				T::WeightInfo::note_author(),608				<T as Config>::WeightInfo::note_author(),
603				DispatchClass::Mandatory,609				DispatchClass::Mandatory,
604			);610			);
605		}611		}
625			let result = Self::assemble_collators(active_candidates);631			let result = Self::assemble_collators(active_candidates);
626632
627			frame_system::Pallet::<T>::register_extra_weight_unchecked(633			frame_system::Pallet::<T>::register_extra_weight_unchecked(
628				T::WeightInfo::new_session(candidates_len_before as u32, removed as u32),634				<T as Config>::WeightInfo::new_session(
635					candidates_len_before as u32,
636					removed as u32,
637				),
modifiedpallets/collator-selection/src/mock.rsdiffbeforeafterboth
--- a/pallets/collator-selection/src/mock.rs
+++ b/pallets/collator-selection/src/mock.rs
@@ -225,6 +225,7 @@
 	type MaxXcmAllowedLocations = MaxXcmAllowedLocations;
 	type AppPromotionDailyRate = AppPromotionDailyRate;
 	type DayRelayBlocks = DayRelayBlocks;
+	type WeightInfo = ();
 }
 
 ord_parameter_types! {
modifiedpallets/configuration/Cargo.tomldiffbeforeafterboth
--- a/pallets/configuration/Cargo.toml
+++ b/pallets/configuration/Cargo.toml
@@ -12,6 +12,7 @@
 ] }
 frame-support = { default-features = false, git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.36" }
 frame-system = { default-features = false, git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.36" }
+frame-benchmarking = { default-features = false, optional = true, git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.36" }
 sp-runtime = { default-features = false, git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.36" }
 sp-std = { default-features = false, git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.36" }
 sp-core = { default-features = false, git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.36" }
@@ -22,10 +23,12 @@
 
 [features]
 default = ["std"]
+runtime-benchmarks = ["frame-benchmarking"]
 std = [
 	"parity-scale-codec/std",
 	"frame-support/std",
 	"frame-system/std",
+	"frame-benchmarking/std",
 	"sp-runtime/std",
 	"sp-std/std",
 	"sp-core/std",
addedpallets/configuration/src/benchmarking.rsdiffbeforeafterboth
--- /dev/null
+++ b/pallets/configuration/src/benchmarking.rs
@@ -0,0 +1,100 @@
+// Copyright 2019-2022 Unique Network (Gibraltar) Ltd.
+// This file is part of Unique Network.
+
+// Unique Network is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+
+// Unique Network is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+
+// You should have received a copy of the GNU General Public License
+// along with Unique Network. If not, see <http://www.gnu.org/licenses/>.
+
+//! Benchmarking setup for pallet-configuration
+
+use super::*;
+use frame_benchmarking::benchmarks;
+use frame_system::{EventRecord, RawOrigin};
+use frame_support::{assert_ok, BoundedVec, traits::Currency};
+use xcm::v1::MultiLocation;
+
+fn assert_last_event<T: Config>(generic_event: <T as Config>::RuntimeEvent) {
+	let events = frame_system::Pallet::<T>::events();
+	let system_event: <T as frame_system::Config>::RuntimeEvent = generic_event.into();
+	// compare to the last event record
+	let EventRecord { event, .. } = &events[events.len() - 1];
+	assert_eq!(event, &system_event);
+}
+
+benchmarks! {
+	where_clause { where T: Config }
+
+	set_weight_to_fee_coefficient_override {
+		let coeff: u64 = 999;
+	}: {
+		assert_ok!(
+			<Pallet<T>>::set_weight_to_fee_coefficient_override(RawOrigin::Root.into(), Some(coeff))
+		);
+	}
+
+	set_min_gas_price_override {
+		let coeff: u64 = 999;
+	}: {
+		assert_ok!(
+			<Pallet<T>>::set_min_gas_price_override(RawOrigin::Root.into(), Some(coeff))
+		);
+	}
+
+	set_xcm_allowed_locations {
+		let locations: BoundedVec<MultiLocation, T::MaxXcmAllowedLocations> = Default::default();
+	}: {
+		assert_ok!(
+			<Pallet<T>>::set_xcm_allowed_locations(RawOrigin::Root.into(), Some(locations))
+		);
+	}
+
+	set_app_promotion_configuration_override {
+		let configuration: AppPromotionConfiguration<T::BlockNumber> = Default::default();
+	}: {
+		assert_ok!(
+			<Pallet<T>>::set_app_promotion_configuration_override(RawOrigin::Root.into(), configuration)
+		);
+	}
+
+	set_collator_selection_desired_collators {
+		let max: u32 = 999;
+	}: {
+		assert_ok!(
+			<Pallet<T>>::set_collator_selection_desired_collators(RawOrigin::Root.into(), Some(max.clone()))
+		);
+	}
+	verify {
+		assert_last_event::<T>(Event::NewDesiredCollators{desired_collators: Some(max)}.into());
+	}
+
+	set_collator_selection_license_bond {
+		let bond_cost: Option<BalanceOf<T>> = Some(T::Currency::minimum_balance() * 10u32.into());
+	}: {
+		assert_ok!(
+			<Pallet<T>>::set_collator_selection_license_bond(RawOrigin::Root.into(), bond_cost.clone())
+		);
+	}
+	verify {
+		assert_last_event::<T>(Event::NewCollatorLicenseBond{bond_cost}.into());
+	}
+
+	set_collator_selection_kick_threshold {
+		let threshold: Option<T::BlockNumber> = Some(900u32.into());
+	}: {
+		assert_ok!(
+			<Pallet<T>>::set_collator_selection_kick_threshold(RawOrigin::Root.into(), threshold.clone())
+		);
+	}
+	verify {
+		assert_last_event::<T>(Event::NewCollatorKickThreshold{length_in_blocks: threshold}.into());
+	}
+}
modifiedpallets/configuration/src/lib.rsdiffbeforeafterboth
--- a/pallets/configuration/src/lib.rs
+++ b/pallets/configuration/src/lib.rs
@@ -34,6 +34,10 @@
 pub use pallet::*;
 use sp_core::U256;
 
+#[cfg(feature = "runtime-benchmarks")]
+mod benchmarking;
+pub mod weights;
+
 #[pallet]
 mod pallet {
 	use super::*;
@@ -45,6 +49,7 @@
 	use frame_system::{pallet_prelude::OriginFor, ensure_root, Config as SystemConfig};
 	use xcm::v1::MultiLocation;
 
+	pub use crate::weights::WeightInfo;
 	pub type BalanceOf<T> =
 		<<T as Config>::Currency as Currency<<T as SystemConfig>::AccountId>>::Balance;
 
@@ -74,6 +79,9 @@
 		type DefaultCollatorSelectionLicenseBond: Get<BalanceOf<Self>>;
 		#[pallet::constant]
 		type DefaultCollatorSelectionKickThreshold: Get<Self::BlockNumber>;
+
+		/// The weight information of this pallet.
+		type WeightInfo: WeightInfo;
 	}
 
 	#[pallet::event]
@@ -140,7 +148,7 @@
 	#[pallet::call]
 	impl<T: Config> Pallet<T> {
 		#[pallet::call_index(0)]
-		#[pallet::weight(T::DbWeight::get().writes(1))]
+		#[pallet::weight(T::WeightInfo::set_weight_to_fee_coefficient_override())]
 		pub fn set_weight_to_fee_coefficient_override(
 			origin: OriginFor<T>,
 			coeff: Option<u64>,
@@ -155,7 +163,7 @@
 		}
 
 		#[pallet::call_index(1)]
-		#[pallet::weight(T::DbWeight::get().writes(1))]
+		#[pallet::weight(T::WeightInfo::set_min_gas_price_override())]
 		pub fn set_min_gas_price_override(
 			origin: OriginFor<T>,
 			coeff: Option<u64>,
@@ -170,7 +178,7 @@
 		}
 
 		#[pallet::call_index(2)]
-		#[pallet::weight(T::DbWeight::get().writes(1))]
+		#[pallet::weight(T::WeightInfo::set_xcm_allowed_locations())]
 		pub fn set_xcm_allowed_locations(
 			origin: OriginFor<T>,
 			locations: Option<BoundedVec<MultiLocation, T::MaxXcmAllowedLocations>>,
@@ -181,7 +189,7 @@
 		}
 
 		#[pallet::call_index(3)]
-		#[pallet::weight(T::DbWeight::get().writes(1))]
+		#[pallet::weight(T::WeightInfo::set_app_promotion_configuration_override())]
 		pub fn set_app_promotion_configuration_override(
 			origin: OriginFor<T>,
 			mut configuration: AppPromotionConfiguration<T::BlockNumber>,
@@ -202,7 +210,7 @@
 		}
 
 		#[pallet::call_index(4)]
-		#[pallet::weight(T::DbWeight::get().writes(1))]
+		#[pallet::weight(T::WeightInfo::set_collator_selection_desired_collators())]
 		pub fn set_collator_selection_desired_collators(
 			origin: OriginFor<T>,
 			max: Option<u32>,
@@ -224,7 +232,7 @@
 		}
 
 		#[pallet::call_index(5)]
-		#[pallet::weight(T::DbWeight::get().writes(1))]
+		#[pallet::weight(T::WeightInfo::set_collator_selection_license_bond())]
 		pub fn set_collator_selection_license_bond(
 			origin: OriginFor<T>,
 			amount: Option<BalanceOf<T>>,
@@ -240,7 +248,7 @@
 		}
 
 		#[pallet::call_index(6)]
-		#[pallet::weight(T::DbWeight::get().writes(1))]
+		#[pallet::weight(T::WeightInfo::set_collator_selection_kick_threshold())]
 		pub fn set_collator_selection_kick_threshold(
 			origin: OriginFor<T>,
 			threshold: Option<T::BlockNumber>,
addedpallets/configuration/src/weights.rsdiffbeforeafterboth
--- /dev/null
+++ b/pallets/configuration/src/weights.rs
@@ -0,0 +1,123 @@
+// Template adopted from https://github.com/paritytech/substrate/blob/master/.maintain/frame-weight-template.hbs
+
+//! Autogenerated weights for pallet_configuration
+//!
+//! THIS FILE WAS AUTO-GENERATED USING THE SUBSTRATE BENCHMARK CLI VERSION 4.0.0-dev
+//! DATE: 2022-12-28, STEPS: `50`, REPEAT: 80, LOW RANGE: `[]`, HIGH RANGE: `[]`
+//! EXECUTION: None, WASM-EXECUTION: Compiled, CHAIN: None, DB CACHE: 1024
+
+// Executed Command:
+// target/release/unique-collator
+// benchmark
+// pallet
+// --pallet
+// pallet-configuration
+// --wasm-execution
+// compiled
+// --extrinsic
+// *
+// --template
+// .maintain/frame-weight-template.hbs
+// --steps=50
+// --repeat=80
+// --heap-pages=4096
+// --output=./pallets/configuration/src/weights.rs
+
+#![cfg_attr(rustfmt, rustfmt_skip)]
+#![allow(unused_parens)]
+#![allow(unused_imports)]
+#![allow(missing_docs)]
+#![allow(clippy::unnecessary_cast)]
+
+use frame_support::{traits::Get, weights::{Weight, constants::RocksDbWeight}};
+use sp_std::marker::PhantomData;
+
+/// Weight functions needed for pallet_configuration.
+pub trait WeightInfo {
+	fn set_weight_to_fee_coefficient_override() -> Weight;
+	fn set_min_gas_price_override() -> Weight;
+	fn set_xcm_allowed_locations() -> Weight;
+	fn set_app_promotion_configuration_override() -> Weight;
+	fn set_collator_selection_desired_collators() -> Weight;
+	fn set_collator_selection_license_bond() -> Weight;
+	fn set_collator_selection_kick_threshold() -> Weight;
+}
+
+/// Weights for pallet_configuration using the Substrate node and recommended hardware.
+pub struct SubstrateWeight<T>(PhantomData<T>);
+impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
+	// Storage: Configuration WeightToFeeCoefficientOverride (r:0 w:1)
+	fn set_weight_to_fee_coefficient_override() -> Weight {
+		Weight::from_ref_time(5_691_000 as u64)
+			.saturating_add(T::DbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration MinGasPriceOverride (r:0 w:1)
+	fn set_min_gas_price_override() -> Weight {
+		Weight::from_ref_time(5_521_000 as u64)
+			.saturating_add(T::DbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration XcmAllowedLocationsOverride (r:0 w:1)
+	fn set_xcm_allowed_locations() -> Weight {
+		Weight::from_ref_time(6_091_000 as u64)
+			.saturating_add(T::DbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration AppPromomotionConfigurationOverride (r:0 w:1)
+	fn set_app_promotion_configuration_override() -> Weight {
+		Weight::from_ref_time(6_241_000 as u64)
+			.saturating_add(T::DbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration CollatorSelectionDesiredCollatorsOverride (r:0 w:1)
+	fn set_collator_selection_desired_collators() -> Weight {
+		Weight::from_ref_time(25_298_000 as u64)
+			.saturating_add(T::DbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration CollatorSelectionLicenseBondOverride (r:0 w:1)
+	fn set_collator_selection_license_bond() -> Weight {
+		Weight::from_ref_time(18_675_000 as u64)
+			.saturating_add(T::DbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration CollatorSelectionKickThresholdOverride (r:0 w:1)
+	fn set_collator_selection_kick_threshold() -> Weight {
+		Weight::from_ref_time(18_044_000 as u64)
+			.saturating_add(T::DbWeight::get().writes(1 as u64))
+	}
+}
+
+// For backwards compatibility and tests
+impl WeightInfo for () {
+	// Storage: Configuration WeightToFeeCoefficientOverride (r:0 w:1)
+	fn set_weight_to_fee_coefficient_override() -> Weight {
+		Weight::from_ref_time(5_691_000 as u64)
+			.saturating_add(RocksDbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration MinGasPriceOverride (r:0 w:1)
+	fn set_min_gas_price_override() -> Weight {
+		Weight::from_ref_time(5_521_000 as u64)
+			.saturating_add(RocksDbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration XcmAllowedLocationsOverride (r:0 w:1)
+	fn set_xcm_allowed_locations() -> Weight {
+		Weight::from_ref_time(6_091_000 as u64)
+			.saturating_add(RocksDbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration AppPromomotionConfigurationOverride (r:0 w:1)
+	fn set_app_promotion_configuration_override() -> Weight {
+		Weight::from_ref_time(6_241_000 as u64)
+			.saturating_add(RocksDbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration CollatorSelectionDesiredCollatorsOverride (r:0 w:1)
+	fn set_collator_selection_desired_collators() -> Weight {
+		Weight::from_ref_time(25_298_000 as u64)
+			.saturating_add(RocksDbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration CollatorSelectionLicenseBondOverride (r:0 w:1)
+	fn set_collator_selection_license_bond() -> Weight {
+		Weight::from_ref_time(18_675_000 as u64)
+			.saturating_add(RocksDbWeight::get().writes(1 as u64))
+	}
+	// Storage: Configuration CollatorSelectionKickThresholdOverride (r:0 w:1)
+	fn set_collator_selection_kick_threshold() -> Weight {
+		Weight::from_ref_time(18_044_000 as u64)
+			.saturating_add(RocksDbWeight::get().writes(1 as u64))
+	}
+}
modifiedruntime/common/config/pallets/mod.rsdiffbeforeafterboth
--- a/runtime/common/config/pallets/mod.rs
+++ b/runtime/common/config/pallets/mod.rs
@@ -121,6 +121,7 @@
 	type MaxXcmAllowedLocations = ConstU32<16>;
 	type AppPromotionDailyRate = AppPromotionDailyRate;
 	type DayRelayBlocks = DayRelayBlocks;
+	type WeightInfo = pallet_configuration::weights::SubstrateWeight<Self>;
 }
 
 impl pallet_maintenance::Config for Runtime {
modifiedruntime/common/construct_runtime/mod.rsdiffbeforeafterboth
--- a/runtime/common/construct_runtime/mod.rs
+++ b/runtime/common/construct_runtime/mod.rs
@@ -32,17 +32,17 @@
                 ParachainSystem: cumulus_pallet_parachain_system::{Pallet, Call, Config, Storage, Inherent, Event<T>, ValidateUnsigned} = 20,
                 ParachainInfo: parachain_info::{Pallet, Storage, Config} = 21,
 
-                Aura: pallet_aura::{Pallet, Config<T>} = 22,
-                AuraExt: cumulus_pallet_aura_ext::{Pallet, Config} = 23,
+                #[runtimes(opal)]
+                Authorship: pallet_authorship::{Pallet, Call, Storage} = 22,
 
                 #[runtimes(opal)]
-                Authorship: pallet_authorship::{Pallet, Call, Storage} = 24,
+                CollatorSelection: pallet_collator_selection::{Pallet, Call, Storage, Event<T>, Config<T>} = 23,
 
                 #[runtimes(opal)]
-                CollatorSelection: pallet_collator_selection::{Pallet, Call, Storage, Event<T>, Config<T>} = 25,
+                Session: pallet_session::{Pallet, Call, Storage, Event, Config<T>} = 24,
 
-                #[runtimes(opal)]
-                Session: pallet_session::{Pallet, Call, Storage, Event, Config<T>} = 26,
+                Aura: pallet_aura::{Pallet, Config<T>} = 25,
+                AuraExt: cumulus_pallet_aura_ext::{Pallet, Config} = 26,
 
                 Balances: pallet_balances::{Pallet, Call, Storage, Config<T>, Event<T>} = 30,
                 RandomnessCollectiveFlip: pallet_randomness_collective_flip::{Pallet, Storage} = 31,
modifiedruntime/common/runtime_apis.rsdiffbeforeafterboth
--- a/runtime/common/runtime_apis.rs
+++ b/runtime/common/runtime_apis.rs
@@ -686,6 +686,7 @@
                     list_benchmark!(list, extra, pallet_unique, Unique);
                     list_benchmark!(list, extra, pallet_structure, Structure);
                     list_benchmark!(list, extra, pallet_inflation, Inflation);
+                    list_benchmark!(list, extra, pallet_configuration, Configuration);
 
                     #[cfg(feature = "app-promotion")]
                     list_benchmark!(list, extra, pallet_app_promotion, AppPromotion);
@@ -755,6 +756,7 @@
                     add_benchmark!(params, batches, pallet_unique, Unique);
                     add_benchmark!(params, batches, pallet_structure, Structure);
                     add_benchmark!(params, batches, pallet_inflation, Inflation);
+                    add_benchmark!(params, batches, pallet_configuration, Configuration);
 
                     #[cfg(feature = "app-promotion")]
                     add_benchmark!(params, batches, pallet_app_promotion, AppPromotion);
modifiedruntime/opal/Cargo.tomldiffbeforeafterboth
--- a/runtime/opal/Cargo.toml
+++ b/runtime/opal/Cargo.toml
@@ -28,6 +28,7 @@
     'pallet-evm-coder-substrate/runtime-benchmarks',
     'pallet-balances/runtime-benchmarks',
     'pallet-timestamp/runtime-benchmarks',
+    'pallet-configuration/runtime-benchmarks',
     'pallet-common/runtime-benchmarks',
     'pallet-structure/runtime-benchmarks',
     'pallet-fungible/runtime-benchmarks',
modifiedruntime/quartz/Cargo.tomldiffbeforeafterboth
--- a/runtime/quartz/Cargo.toml
+++ b/runtime/quartz/Cargo.toml
@@ -28,6 +28,7 @@
     'pallet-evm-coder-substrate/runtime-benchmarks',
     'pallet-balances/runtime-benchmarks',
     'pallet-timestamp/runtime-benchmarks',
+    'pallet-configuration/runtime-benchmarks',
     'pallet-common/runtime-benchmarks',
     'pallet-structure/runtime-benchmarks',
     'pallet-fungible/runtime-benchmarks',
modifiedruntime/unique/Cargo.tomldiffbeforeafterboth
--- a/runtime/unique/Cargo.toml
+++ b/runtime/unique/Cargo.toml
@@ -28,6 +28,7 @@
     'pallet-evm-coder-substrate/runtime-benchmarks',
     'pallet-balances/runtime-benchmarks',
     'pallet-timestamp/runtime-benchmarks',
+    'pallet-configuration/runtime-benchmarks',
     'pallet-common/runtime-benchmarks',
     'pallet-structure/runtime-benchmarks',
     'pallet-fungible/runtime-benchmarks',