Skip to content

[pallet-broker] add extrinsic to force transfer a region #10856

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
mertwole wants to merge 8 commits into
base: master
Choose a base branch
from
Open

[pallet-broker] add extrinsic to force transfer a region #10856

mertwole wants to merge 8 commits into from

Conversation

@mertwole
Copy link
Contributor

@mertwole mertwole commented Jan 21, 2026 •
edited
Loading

Part of #6929

Add an extrinsic to the pallet-broker allowing to forcefully transfer a region, ignoring the owner.

@mertwole mertwole marked this pull request as ready for review January 21, 2026 10:27
@mertwole mertwole requested a review from a team as a code owner January 21, 2026 10:27
@mertwole
Copy link
Contributor Author

mertwole commented Jan 21, 2026

/cmd bench --pallet pallet_broker

@mertwole mertwole requested a review from seadanda January 21, 2026 10:28
@github-actions
Copy link
Contributor

github-actions bot commented Jan 21, 2026

Command "bench --pallet pallet_broker" has started 🚀 See logs here

@github-actions
Copy link
Contributor

github-actions bot commented Jan 21, 2026

Command "bench --pallet pallet_broker" has finished ✅ See logs here

Details

Subweight results:
File Extrinsic Old New Change [%]
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs on_new_timeslice 30.25us 76.61us +153.31
substrate/frame/broker/src/weights.rs request_core_count 3.99us 6.32us +58.35
substrate/frame/broker/src/weights.rs on_new_timeslice 251.00ns 364.00ns +45.02
substrate/frame/broker/src/weights.rs request_revenue_info_at 184.00ns 219.00ns +19.02
substrate/frame/broker/src/weights.rs purchase_credit 171.45us 55.20us -67.80
substrate/frame/broker/src/weights.rs drop_history 261.46us 79.73us -69.51
substrate/frame/broker/src/weights.rs drop_contribution 226.35us 59.32us -73.79
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs drop_history 328.00us 83.11us -74.66
substrate/frame/broker/src/weights.rs drop_region 186.28us 46.32us -75.13
substrate/frame/broker/src/weights.rs drop_renewal 191.67us 47.19us -75.38
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs purchase_credit 371.54us 90.03us -75.77
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs drop_renewal 205.11us 49.21us -76.01
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs drop_region 218.23us 47.74us -78.12
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs drop_contribution 311.26us 68.09us -78.12
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs reserve 150.27us 30.03us -80.01
substrate/frame/broker/src/weights.rs partition 242.93us 47.27us -80.54
substrate/frame/broker/src/weights.rs reserve 140.42us 27.17us -80.65
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs partition 248.38us 48.01us -80.67
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs unreserve 149.70us 28.27us -81.11
substrate/frame/broker/src/weights.rs unreserve 139.62us 25.68us -81.61
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs assign 336.47us 61.75us -81.65
substrate/frame/broker/src/weights.rs disable_auto_renew 140.21us 25.54us -81.79
substrate/frame/broker/src/weights.rs assign 332.04us 60.15us -81.88
substrate/frame/broker/src/weights.rs purchase 287.54us 51.81us -81.98
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs renew 653.89us 111.20us -82.99
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs request_core_count 196.49us 33.34us -83.03
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs transfer 146.75us 24.39us -83.38
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs purchase 456.78us 75.26us -83.52
substrate/frame/broker/src/weights.rs transfer 141.40us 23.20us -83.59
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs remove_assignment 146.87us 23.96us -83.68
substrate/frame/broker/src/weights.rs remove_assignment 140.78us 22.37us -84.11
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs disable_auto_renew 143.78us 21.52us -85.03
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs interlace 349.67us 49.98us -85.71
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs enable_auto_renew 911.33us 129.74us -85.76
substrate/frame/broker/src/weights.rs interlace 344.21us 48.69us -85.86
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs force_reserve 417.83us 54.86us -86.87
substrate/frame/broker/src/weights.rs renew 572.33us 74.58us -86.97
substrate/frame/broker/src/weights.rs set_lease 132.71us 17.18us -87.06
substrate/frame/broker/src/weights.rs claim_revenue 725.23us 93.76us -87.07
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs process_revenue 550.62us 69.99us -87.29
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs request_revenue_info_at 191.39us 23.93us -87.50
substrate/frame/broker/src/weights.rs remove_lease 132.34us 16.30us -87.69
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs claim_revenue 728.04us 89.08us -87.76
substrate/frame/broker/src/weights.rs process_revenue 414.96us 50.27us -87.88
substrate/frame/broker/src/weights.rs force_reserve 403.07us 47.13us -88.31
substrate/frame/broker/src/weights.rs enable_auto_renew 834.47us 97.31us -88.34
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs process_core_schedule 457.11us 47.63us -89.58
substrate/frame/broker/src/weights.rs swap_leases 129.63us 12.51us -90.35
substrate/frame/broker/src/weights.rs process_core_schedule 262.12us 23.52us -91.03
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs remove_lease 135.72us 12.18us -91.03
substrate/frame/broker/src/weights.rs do_tick_base 210.65us 18.01us -91.45
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs set_lease 188.45us 15.51us -91.77
substrate/frame/broker/src/weights.rs process_pool 130.31us 10.63us -91.85
substrate/frame/broker/src/weights.rs process_core_count 131.08us 10.38us -92.08
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs process_core_count 133.45us 9.93us -92.56
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs pool 668.18us 48.47us -92.75
substrate/frame/broker/src/weights.rs pool 664.63us 46.56us -92.99
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs process_pool 133.15us 9.24us -93.06
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs do_tick_base 239.51us 15.99us -93.32
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs swap_leases 132.63us 8.14us -93.86
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs start_sales 3.00ms 141.11us -95.30
substrate/frame/broker/src/weights.rs start_sales 1.83ms 79.16us -95.67
substrate/frame/broker/src/weights.rs configure 102.15us 3.61us -96.47
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs configure 102.79us 3.62us -96.47
substrate/frame/broker/src/weights.rs notify_revenue 102.07us 3.48us -96.60
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs notify_revenue 102.68us 3.38us -96.70
substrate/frame/broker/src/weights.rs notify_core_count 101.91us 3.27us -96.79
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs notify_core_count 102.57us 3.20us -96.88
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs rotate_sale 108.90ms 2.90ms -97.33
substrate/frame/broker/src/weights.rs rotate_sale 105.70ms 2.11ms -98.00
substrate/frame/broker/src/weights.rs force_transfer 23.00us Added
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs force_transfer 23.50us Added
Command output:

✅ Successful benchmarks of runtimes/pallets:
-- dev: ['pallet_broker']
-- coretime-westend: ['pallet_broker']

@mertwole mertwole added the T2-pallets This PR/Issue is related to a particular pallet. label Jan 22, 2026
@mertwole mertwole requested review from bkchr and ggwpez January 26, 2026 09:40
muharem
muharem reviewed Jan 29, 2026
View reviewed changes
Copy link
Contributor

@muharem muharem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good. just the benchmark results comment

substrate/frame/broker/src/weights.rs
// Measured: `496`
// Estimated: `0`
// Minimum execution time: 22_033_000 picoseconds.
Weight::from_parts(22_996_000, 0)
Copy link
Contributor

@muharem muharem Jan 29, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there should be some reads and writes. and I assume your even assertion passes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@muharem muharem muharem left review comments

@seadanda seadanda Awaiting requested review from seadanda

@bkchr bkchr Awaiting requested review from bkchr

@ggwpez ggwpez Awaiting requested review from ggwpez

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

T2-pallets This PR/Issue is related to a particular pallet.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mertwole @muharem