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

Restore Rococo <> Wococo bridges (headers + messages)#4452

Merged
paritytech-processbot[bot] merged 5 commits intomasterfrom
restore-rococo-wococo-bridge
Dec 7, 2021
Merged

Restore Rococo <> Wococo bridges (headers + messages)#4452
paritytech-processbot[bot] merged 5 commits intomasterfrom
restore-rococo-wococo-bridge

Conversation

@svyatonik
Copy link
Copy Markdown
Contributor

May be tested with scripts in https://github.com/svyatonik/rococo-wococo.test and updated relayer from paritytech/parity-bridges-common#1241 . This version:

  1. doesn't include any message filters;
  2. needs message relayers to be running in altruistic mode, because R/W tokens has no value;
  3. requires relayers fund account creation on R/W once this goes live;
  4. since we have been running R<>W headers bridge for a while and I believe Rococo hasn't been restarted since that, we don't need to (re)initialize headers bridge. Instead we shall run standalone headers relay for a while && let it sync missing headers.

@svyatonik svyatonik 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. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. labels Dec 2, 2021
Copy link
Copy Markdown
Contributor

@tomusdrw tomusdrw left a comment

Choose a reason for hiding this comment

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

since we have been running R<>W headers bridge for a while and I believe Rococo hasn't been restarted since that, we don't need to (re)initialize headers bridge. Instead we shall run standalone headers relay for a while && let it sync missing headers.

Actually it was restarted and the header relayers are not running since then. We need to initialize it from scratch and I think it's best to run the complex relayer right from scratch.

@svyatonik
Copy link
Copy Markdown
Contributor Author

bot merge

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

Labels

A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants