Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Add force_open_hrmp_channel Call #6155

Merged
paritytech-processbot merged 6 commits into from
Oct 17, 2022
Merged

Add force_open_hrmp_channel Call #6155

paritytech-processbot merged 6 commits into from
Oct 17, 2022

Conversation

@joepetrowski
Copy link
Contributor

@joepetrowski joepetrowski commented Oct 14, 2022

Open an HRMP channel using the Root origin. This will avoid the nested messages referenda needed to establish channels with common good chains.

Supersedes #4998
Closes #4648

@joepetrowski joepetrowski added A0-please_review Pull request needs code review. B7-runtimenoteworthy C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. labels Oct 14, 2022
@joepetrowski
Copy link
Contributor Author

joepetrowski commented Oct 14, 2022

/cmd queue -c bench-bot $ pallet westend-dev polkadot-runtime-parachains::hrmp

@command-bot
Copy link

command-bot bot commented Oct 14, 2022 •
edited
Loading

@joepetrowski https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951028 was started for your command "$PIPELINE_SCRIPTS_DIR/bench-bot.sh" pallet westend-dev polkadot-runtime-parachains::hrmp. Check out https://gitlab.parity.io/parity/mirrors/polkadot/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment /cmd cancel 14-35bb0990-2f21-4964-8df5-c817cc2c6263 to cancel this command or /cmd cancel to cancel all commands in this pull request.

@joepetrowski joepetrowski mentioned this pull request Oct 14, 2022
Closed
@command-bot
Copy link

command-bot bot commented Oct 14, 2022

@joepetrowski Command "$PIPELINE_SCRIPTS_DIR/bench-bot.sh" pallet westend-dev polkadot-runtime-parachains::hrmp has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951028 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951028/artifacts/download.

@ggwpez
Add HRMP benches to Polkadot
6892d6f 
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@paritytech-ci paritytech-ci requested review from a team October 14, 2022 09:44
@ggwpez
Copy link
Member

ggwpez commented Oct 14, 2022

/cmd queue -c bench-bot $ pallet westend-dev runtime_parachains::hrmp
/cmd queue -c bench-bot $ pallet kusama-dev runtime_parachains::hrmp

@command-bot
Copy link

command-bot bot commented Oct 14, 2022 •
edited
Loading

@ggwpez https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951410 was started for your command "$PIPELINE_SCRIPTS_DIR/bench-bot.sh" pallet westend-dev runtime_parachains::hrmp. Check out https://gitlab.parity.io/parity/mirrors/polkadot/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment /cmd cancel 15-2d123fc7-cb22-4e7a-a7d5-5365a4bc23b9 to cancel this command or /cmd cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Oct 14, 2022 •
edited
Loading

@ggwpez https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951411 was started for your command "$PIPELINE_SCRIPTS_DIR/bench-bot.sh" pallet kusama-dev runtime_parachains::hrmp. Check out https://gitlab.parity.io/parity/mirrors/polkadot/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment /cmd cancel 16-d8a9093b-1baf-4eff-a932-f52380099a00 to cancel this command or /cmd cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Oct 14, 2022

@ggwpez Command "$PIPELINE_SCRIPTS_DIR/bench-bot.sh" pallet westend-dev runtime_parachains::hrmp has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951410 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951410/artifacts/download.

@command-bot
Copy link

command-bot bot commented Oct 14, 2022

@ggwpez Command "$PIPELINE_SCRIPTS_DIR/bench-bot.sh" pallet kusama-dev runtime_parachains::hrmp has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951411 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951411/artifacts/download.

@ggwpez
Copy link
Member

ggwpez commented Oct 14, 2022

/cmd queue -c bench-bot $ runtime westend-dev runtime_parachains::hrmp

@command-bot
Copy link

command-bot bot commented Oct 14, 2022 •
edited
Loading

@ggwpez https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951412 was started for your command "$PIPELINE_SCRIPTS_DIR/bench-bot.sh" runtime westend-dev runtime_parachains::hrmp. Check out https://gitlab.parity.io/parity/mirrors/polkadot/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment /cmd cancel 17-20e870c8-0345-4ee6-adad-79ee8dca9299 to cancel this command or /cmd cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Oct 14, 2022

@ggwpez Command "$PIPELINE_SCRIPTS_DIR/bench-bot.sh" runtime westend-dev runtime_parachains::hrmp has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951412 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/1951412/artifacts/download.

@joepetrowski
Copy link
Contributor Author

joepetrowski commented Oct 17, 2022

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 9d4e1a1 into master Oct 17, 2022
@paritytech-processbot paritytech-processbot bot deleted the joe-root-hrmp-channel-establishment branch October 17, 2022 11:38
ordian pushed a commit that referenced this pull request Oct 21, 2022
Merge branch 'master' into giles-common-version
ecd7b49 
* master: (21 commits)
  try and fix build (#6170)
  Companion for EPM duplicate submissions (#6115)
  Bump docker/setup-buildx-action from 2.0.0 to 2.1.0 (#6141)
  companion for #12212 (#6162)
  Bump substrate (#6164)
  BlockId removal: refactor: StorageProvider (#6160)
  availability-recovery: use `IfDisconnected::TryConnect` for chunks (#6081)
  Update clap to version 4 (#6128)
  Add `force_open_hrmp_channel` Call (#6155)
  Fix fuzzing builds xcm-fuzz and erasure-coding fuzzer (#6153)
  BlockId removal refactor: Backend::state_at (#6149)
  First round of implementers guide fixes (#6146)
  bump zombienet version (#6142)
  lingua.dic is not managed by CI team (#6148)
  pallet-mmr: RPC and Runtime APIs work with block numbers (#6072)
  Separate preparation timeouts for PVF prechecking and execution (#6139)
  Malus: add disputed block percentage (#6100)
  refactor grid topology to expose more info to subsystems (#6140)
  Manual Para Lock (#5451)
  Expose node subcommands in Malus CLI (#6135)
  ...
@Polkadot-Forum
Copy link

Polkadot-Forum commented Nov 9, 2022

This pull request has been mentioned on Polkadot Forum. There might be relevant details there:

https://forum.polkadot.network/t/polkadot-release-analysis/1026/2

@github-actions github-actions bot mentioned this pull request Nov 10, 2022
Closed
@github-actions github-actions bot mentioned this pull request Nov 23, 2022
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@pepyakin pepyakin Awaiting requested review from pepyakin

5 more reviewers

@gavofyork gavofyork gavofyork approved these changes

@ordian ordian ordian approved these changes

@bkchr bkchr bkchr approved these changes

@muharem muharem muharem approved these changes

@ggwpez ggwpez ggwpez approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Root call to open / accept / establish HRMP channel

7 participants

@joepetrowski @ggwpez @Polkadot-Forum @gavofyork @ordian @bkchr @muharem