Skip to content

[pallet-broker] add extrinsic to forcefully remove the potential renewal#10828

Merged
mertwole merged 19 commits into
masterfrom
mertwole-add-remove-potential-renewal
Jan 22, 2026
Merged

[pallet-broker] add extrinsic to forcefully remove the potential renewal#10828
mertwole merged 19 commits into
masterfrom
mertwole-add-remove-potential-renewal

Conversation

@mertwole

Copy link
Copy Markdown
Contributor

Part of #6929

Add an extrinsic allowing to forcefully remove the existing potential renewal from chain without the need to directly manipulate the storage.

@mertwole mertwole requested a review from a team as a code owner January 16, 2026 11:07
@mertwole mertwole added the T2-pallets This PR/Issue is related to a particular pallet. label Jan 16, 2026
@mertwole

Copy link
Copy Markdown
Contributor Author

/cmd prdoc --audience runtime_user --bump patch

@mertwole

Copy link
Copy Markdown
Contributor Author

bot bench substrate-pallet --pallet=pallet_broker
bot bench cumulus-coretime --pallet=pallet_broker --runtime=coretime-rococo
bot bench cumulus-coretime --pallet=pallet_broker --runtime=coretime-westend

@command-bot

command-bot Bot commented Jan 16, 2026

Copy link
Copy Markdown

"$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" --subcommand=pallet --runtime=dev --target_dir=substrate --features=riscv --pallet=pallet_broker was queued.

Comment bot cancel 1-2b061a0f-29b1-4ab7-99de-e5c7ca19f919 to cancel this command or bot cancel to cancel all commands in this pull request.

@github-actions

Copy link
Copy Markdown
Contributor

We have migrated the command bot to GHA

Please, see the new usage instructions here or here. Soon the old commands will be disabled.

@command-bot

command-bot Bot commented Jan 16, 2026

Copy link
Copy Markdown

"$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" --subcommand=pallet --runtime=coretime-rococo --runtime_dir=coretime --target_dir=cumulus --pallet=pallet_broker was queued.

Comment bot cancel 2-fa510f13-dd71-4a75-b282-b6150c5a3817 to cancel this command or bot cancel to cancel all commands in this pull request.

@command-bot

command-bot Bot commented Jan 16, 2026

Copy link
Copy Markdown

"$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" --subcommand=pallet --runtime=coretime-westend --runtime_dir=coretime --target_dir=cumulus --pallet=pallet_broker was queued.

Comment bot cancel 3-57023f36-c579-4a23-b0f9-4b57266becfa to cancel this command or bot cancel to cancel all commands in this pull request.

@command-bot

command-bot Bot commented Jan 16, 2026

Copy link
Copy Markdown

@mertwole Command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" --subcommand=pallet --runtime=dev --target_dir=substrate --features=riscv --pallet=pallet_broker has finished. Result:

Error: fatal: unable to access 'https://gitlab.parity.io/parity/mirrors/polkadot-sdk.git/': Could not resolve host: gitlab.parity.io
Error: fatal: unable to access 'https://gitlab.parity.io/parity/mirrors/polkadot-sdk.git/': Could not resolve host: gitlab.parity.io
    at ChildProcess.<anonymous> (/app/build/shell.js:73:39)
    at ChildProcess.emit (node:events:517:28)
    at ChildProcess.emit (node:domain:489:12)
    at maybeClose (node:internal/child_process:1098:16)
    at ChildProcess._handle.onexit (node:internal/child_process:303:5)

@command-bot

command-bot Bot commented Jan 16, 2026

Copy link
Copy Markdown

@mertwole Command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" --subcommand=pallet --runtime=coretime-rococo --runtime_dir=coretime --target_dir=cumulus --pallet=pallet_broker has finished. Result:

Error: fatal: unable to access 'https://gitlab.parity.io/parity/mirrors/polkadot-sdk.git/': Could not resolve host: gitlab.parity.io
Error: fatal: unable to access 'https://gitlab.parity.io/parity/mirrors/polkadot-sdk.git/': Could not resolve host: gitlab.parity.io
    at ChildProcess.<anonymous> (/app/build/shell.js:73:39)
    at ChildProcess.emit (node:events:517:28)
    at ChildProcess.emit (node:domain:489:12)
    at maybeClose (node:internal/child_process:1098:16)
    at ChildProcess._handle.onexit (node:internal/child_process:303:5)

@command-bot

command-bot Bot commented Jan 16, 2026

Copy link
Copy Markdown

@mertwole Command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" --subcommand=pallet --runtime=coretime-westend --runtime_dir=coretime --target_dir=cumulus --pallet=pallet_broker has finished. Result:

Error: fatal: unable to access 'https://gitlab.parity.io/parity/mirrors/polkadot-sdk.git/': Could not resolve host: gitlab.parity.io
Error: fatal: unable to access 'https://gitlab.parity.io/parity/mirrors/polkadot-sdk.git/': Could not resolve host: gitlab.parity.io
    at ChildProcess.<anonymous> (/app/build/shell.js:73:39)
    at ChildProcess.emit (node:events:517:28)
    at ChildProcess.emit (node:domain:489:12)
    at maybeClose (node:internal/child_process:1098:16)
    at ChildProcess._handle.onexit (node:internal/child_process:303:5)

@mertwole

Copy link
Copy Markdown
Contributor Author

/cmd bench --pallet pallet_broker

@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actions

Copy link
Copy Markdown
Contributor

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

Details

Subweight results:
File Extrinsic Old New Change [%]
substrate/frame/broker/src/weights.rs partition 242.93us 673.72us +177.33
substrate/frame/broker/src/weights.rs interlace 344.21us 775.37us +125.26
substrate/frame/broker/src/weights.rs assign 332.04us 736.98us +121.96
substrate/frame/broker/src/weights.rs request_core_count 3.99us 7.05us +76.76
substrate/frame/broker/src/weights.rs on_new_timeslice 251.00ns 331.00ns +31.87
substrate/frame/broker/src/weights.rs request_revenue_info_at 184.00ns 222.00ns +20.65
substrate/frame/broker/src/weights.rs force_reserve 403.07us 446.89us +10.87
substrate/frame/broker/src/weights.rs unreserve 139.62us 150.96us +8.12
substrate/frame/broker/src/weights.rs drop_history 261.46us 282.19us +7.92
substrate/frame/broker/src/weights.rs disable_auto_renew 140.21us 151.19us +7.83
substrate/frame/broker/src/weights.rs reserve 140.42us 151.36us +7.79
substrate/frame/broker/src/weights.rs set_lease 132.71us 142.26us +7.20
substrate/frame/broker/src/weights.rs remove_lease 132.34us 141.46us +6.89
substrate/frame/broker/src/weights.rs drop_region 186.28us 197.97us +6.28
substrate/frame/broker/src/weights.rs remove_assignment 140.78us 149.60us +6.26
substrate/frame/broker/src/weights.rs swap_leases 129.63us 137.54us +6.10
substrate/frame/broker/src/weights.rs purchase_credit 171.45us 181.82us +6.05
substrate/frame/broker/src/weights.rs transfer 141.40us 149.75us +5.91
substrate/frame/broker/src/weights.rs purchase 287.54us 302.63us +5.25
substrate/frame/broker/src/weights.rs remove_potential_renewal 143.02us Added
Command output:

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

@mertwole

Copy link
Copy Markdown
Contributor Author

/cmd bench --pallet pallet_broker
/cmd bench --pallet pallet_broker --runtime coretime-westend

@github-actions

Copy link
Copy Markdown
Contributor

Command "bench --pallet pallet_broker
/cmd bench --pallet pallet_broker --runtime coretime-westend" has started 🚀 See logs here

@github-actions

Copy link
Copy Markdown
Contributor

Command "bench --pallet pallet_broker
/cmd bench --pallet pallet_broker --runtime coretime-westend" has failed ❌! See logs here

@mertwole

Copy link
Copy Markdown
Contributor Author

/cmd bench --pallet pallet_broker

@mertwole

Copy link
Copy Markdown
Contributor Author

/cmd bench --pallet pallet_broker --runtime coretime-westend

@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actions

Copy link
Copy Markdown
Contributor

Command "bench --pallet pallet_broker --runtime coretime-westend" 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 275.17us +809.81
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs partition 248.38us 672.98us +170.94
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs interlace 349.67us 774.89us +121.61
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs assign 336.47us 738.18us +119.39
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs renew 653.89us 766.19us +17.17
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs enable_auto_renew 911.33us 1.04ms +13.66
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs force_reserve 417.83us 457.49us +9.49
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs drop_region 218.23us 200.43us -8.16
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs drop_history 328.00us 284.06us -13.40
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs process_core_schedule 457.11us 371.34us -18.76
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs drop_contribution 311.26us 243.41us -21.80
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs purchase_credit 371.54us 290.31us -21.86
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs request_core_count 196.49us 108.79us -44.63
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs request_revenue_info_at 191.39us 99.20us -48.17
cumulus/parachains/runtimes/coretime/coretime-westend/src/weights/pallet_broker.rs remove_potential_renewal 139.56us Added
substrate/frame/broker/src/weights.rs remove_potential_renewal 0ps Added
Command output:

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

@github-actions

Copy link
Copy Markdown
Contributor

Command "bench --pallet pallet_broker" has failed ❌! See logs here

Details

Command output:

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

@mertwole

Copy link
Copy Markdown
Contributor Author

/cmd bench --pallet pallet_broker --runtime dev

Comment thread substrate/frame/broker/src/lib.rs Outdated
Comment thread substrate/frame/broker/src/tests.rs Outdated
Comment thread substrate/frame/broker/src/tests.rs Outdated
Comment thread substrate/frame/broker/src/tests.rs Outdated
Comment thread substrate/frame/broker/src/benchmarking.rs
Comment thread substrate/frame/broker/src/benchmarking.rs
@mertwole mertwole requested a review from ggwpez January 21, 2026 09:45
@ggwpez

ggwpez commented Jan 21, 2026

Copy link
Copy Markdown
Member

/cmd clean
PS: looks like this does not work like this anymore 🤔

@github-actions

Copy link
Copy Markdown
Contributor

Command "clean" has started 🚀 See logs here

Comment thread prdoc/pr_10828.prdoc Outdated
Comment thread prdoc/pr_10828.prdoc Outdated
@github-actions

Copy link
Copy Markdown
Contributor

Command "clean" has failed ❌! See logs here

@github-actions github-actions Bot requested a review from ggwpez January 21, 2026 10:31
@github-actions

Copy link
Copy Markdown
Contributor

Review required! Latest push from author must always be reviewed

Comment thread substrate/frame/broker/src/dispatchable_impls.rs Outdated
@github-actions github-actions Bot requested a review from seadanda January 22, 2026 10:28
@seadanda seadanda added this pull request to the merge queue Jan 22, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jan 22, 2026
@mertwole mertwole added this pull request to the merge queue Jan 22, 2026
Merged via the queue into master with commit 38e4741 Jan 22, 2026
238 of 241 checks passed
@mertwole mertwole deleted the mertwole-add-remove-potential-renewal branch January 22, 2026 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants