Skip to content

Commit

Permalink
Fix downward message size issue in HRMP benchmark
Browse files Browse the repository at this point in the history
Increase max_downward_message_size to fix force_clean_hrmp benchmark issue.
Increased to 1024. Previously it was inheriting the tests config of 8, which was
too small for the message struct. 1KiB randomly chosen as a reasonable number
which won't present issues for small changes to the message format.
  • Loading branch information
seadanda committed Sep 22, 2023
1 parent ec1153a commit 23d6491
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions polkadot/runtime/parachains/src/hrmp/benchmarking.rs
Original file line number Diff line number Diff line change
Expand Up @@ -208,6 +208,9 @@ mod benchmarks {
assert_ok!(
Configuration::<T>::set_hrmp_max_parachain_inbound_channels(frame_system::RawOrigin::Root.into(), i + 1)
);
assert_ok!(
Configuration::<T>::set_max_downward_message_size(frame_system::RawOrigin::Root.into(), 1024)
);
// .. and enact it.
Configuration::<T>::initializer_on_new_session(&Shared::<T>::scheduled_session());

Expand Down

0 comments on commit 23d6491

Please sign in to comment.