Skip to content
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.

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

Added Rococo BH <> Rococo Bulletin bridge #2724

Merged
merged 14 commits into from
Dec 14, 2023

Conversation

svyatonik
Copy link
Contributor

@svyatonik svyatonik commented Nov 27, 2023

ref #2547

The idea is to have some test environment before adding this stuff to Polkadot Bridge Hub. People chain (which will be bridged to Bulletin Chain) is being added to Rococo/Westend here: paritytech/polkadot-sdk#2281. @joepetrowski said that we'll be able to deploy another "copy" of bulletin chain to bridge with Rococo, so let's add support for Rococo <> Rococo Bulletin chain bridge then.

TODOs left here:

  • final decision on whether we need another two crates - bp_rococo_bulletin_chain and relay-rococo-bulletin-client. Since we'll be deploying the same runtime, let's probably try to reuse Polkadot Bulletin crates? But it causes confuction - i.e. in Rococo runtime I had to use BridgePolkadotBulletinGrandpa pallet name instead of BridgeRococoBulletinGrandpa, so I'm slowly tending to adding other two crates :/ actually we won't be able to change pallet names in Rococo runtime if we'll keep using Polkadot Bulletin runtime as 1:1 for Rococo Bulletin;
  • test everything locally (at least that we are able to send dummy messages from Rococo BH to Rococo Bulletin) before merging.

@svyatonik svyatonik marked this pull request as draft November 27, 2023 14:12
Copy link
Contributor

@bkontur bkontur left a comment

Choose a reason for hiding this comment

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

I think, so far, so good

@svyatonik svyatonik marked this pull request as ready for review December 14, 2023 12:57
@svyatonik svyatonik merged commit 94c44a7 into polkadot-staging Dec 14, 2023
1 check passed
@svyatonik svyatonik deleted the rococo-bulletin-bridge branch December 14, 2023 14:03
svyatonik added a commit that referenced this pull request Dec 15, 2023
* added Rococo BH <> Rococo Bulletin bridge

* init-bridge support

* allow customising finality-related runtime APIs

* revert me

* use Rococo/BridgeHubRococo pretending to be a Polkadot/BridgeHubPolkadot in Rococo <> RococoBulletin bridge

* Revert "revert me"

This reverts commit 90c598d.

* Revert "allow customising finality-related runtime APIs"

This reverts commit b39c32c.

* fmt

* WITH_BRIDGE_ROCOCO_TO_BULLETIN_MESSAGES_PALLET_INDEX

* regenerate bulletin chain runtime (pallet indices have changed)

* fx WITH_BRIDGE_ROCOCO_TO_BULLETIN_MESSAGES_PALLET_INDEX constant because of latest changes

* also change indices in runtime

* fmt

* clippy
svyatonik added a commit that referenced this pull request Dec 15, 2023
* Added Rococo BH <> Rococo Bulletin bridge (#2724)

* added Rococo BH <> Rococo Bulletin bridge

* init-bridge support

* allow customising finality-related runtime APIs

* revert me

* use Rococo/BridgeHubRococo pretending to be a Polkadot/BridgeHubPolkadot in Rococo <> RococoBulletin bridge

* Revert "revert me"

This reverts commit 90c598d.

* Revert "allow customising finality-related runtime APIs"

This reverts commit b39c32c.

* fmt

* WITH_BRIDGE_ROCOCO_TO_BULLETIN_MESSAGES_PALLET_INDEX

* regenerate bulletin chain runtime (pallet indices have changed)

* fx WITH_BRIDGE_ROCOCO_TO_BULLETIN_MESSAGES_PALLET_INDEX constant because of latest changes

* also change indices in runtime

* fmt

* clippy

* remove no longer valid check from the ensure_weights_are_correct (#2740)

* remove no longer valid check from the ensure_weights_are_correct

* fix compilation

* fmt
serban300 pushed a commit to serban300/parity-bridges-common that referenced this pull request Mar 27, 2024
* added Rococo BH <> Rococo Bulletin bridge

* init-bridge support

* allow customising finality-related runtime APIs

* revert me

* use Rococo/BridgeHubRococo pretending to be a Polkadot/BridgeHubPolkadot in Rococo <> RococoBulletin bridge

* Revert "revert me"

This reverts commit 90c598d.

* Revert "allow customising finality-related runtime APIs"

This reverts commit b39c32c.

* fmt

* WITH_BRIDGE_ROCOCO_TO_BULLETIN_MESSAGES_PALLET_INDEX

* regenerate bulletin chain runtime (pallet indices have changed)

* fx WITH_BRIDGE_ROCOCO_TO_BULLETIN_MESSAGES_PALLET_INDEX constant because of latest changes

* also change indices in runtime

* fmt

* clippy
serban300 pushed a commit to serban300/parity-bridges-common that referenced this pull request Apr 8, 2024
* added Rococo BH <> Rococo Bulletin bridge

* init-bridge support

* allow customising finality-related runtime APIs

* revert me

* use Rococo/BridgeHubRococo pretending to be a Polkadot/BridgeHubPolkadot in Rococo <> RococoBulletin bridge

* Revert "revert me"

This reverts commit 90c598d.

* Revert "allow customising finality-related runtime APIs"

This reverts commit b39c32c.

* fmt

* WITH_BRIDGE_ROCOCO_TO_BULLETIN_MESSAGES_PALLET_INDEX

* regenerate bulletin chain runtime (pallet indices have changed)

* fx WITH_BRIDGE_ROCOCO_TO_BULLETIN_MESSAGES_PALLET_INDEX constant because of latest changes

* also change indices in runtime

* fmt

* clippy
bkontur pushed a commit that referenced this pull request May 7, 2024
* Added Rococo BH <> Rococo Bulletin bridge (#2724)

* added Rococo BH <> Rococo Bulletin bridge

* init-bridge support

* allow customising finality-related runtime APIs

* revert me

* use Rococo/BridgeHubRococo pretending to be a Polkadot/BridgeHubPolkadot in Rococo <> RococoBulletin bridge

* Revert "revert me"

This reverts commit 90c598d.

* Revert "allow customising finality-related runtime APIs"

This reverts commit b39c32c.

* fmt

* WITH_BRIDGE_ROCOCO_TO_BULLETIN_MESSAGES_PALLET_INDEX

* regenerate bulletin chain runtime (pallet indices have changed)

* fx WITH_BRIDGE_ROCOCO_TO_BULLETIN_MESSAGES_PALLET_INDEX constant because of latest changes

* also change indices in runtime

* fmt

* clippy

* remove no longer valid check from the ensure_weights_are_correct (#2740)

* remove no longer valid check from the ensure_weights_are_correct

* fix compilation

* fmt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants