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

Add Snowbridge initialize migration on Westend #5747

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

yrong
Copy link
Contributor

@yrong yrong commented Sep 18, 2024

Description

Fix #5074 which missed the runtime migration to initialize channels of the bridge.

@yrong yrong changed the title Snowbridge migration on westend [backport] Snowbridge on Westend Sep 18, 2024
@yrong yrong marked this pull request as ready for review September 18, 2024 05:05
@claravanstaden
Copy link
Contributor

This is not really a backport, so perhaps change the title to "Add Snowbridge initialize migration on Westend", otherwise the title is misleading.

@yrong yrong changed the title [backport] Snowbridge on Westend Add Snowbridge initialize migration on Westend Sep 18, 2024
@acatangiu acatangiu added T15-bridges This PR/Issue is related to bridges. A4-needs-backport Pull request must be backported to all maintained releases. labels Sep 18, 2024
@yrong
Copy link
Contributor Author

yrong commented Sep 19, 2024

https://github.com/paritytech/polkadot-sdk/actions/runs/10923847845/job/30321556177?pr=5747

check-runtime-migration(bridge-hub-rococo) fails though seems nothing to do with this PR. Tested with try-runtime-cli and it work as expected as follows:

➜  polkadot-sdk git:(snowbridge-westend-migration) ✗ RUST_LOG=ethereum_system try-runtime --runtime ./target/release/wbuild/bridge-hub-westend-runtime/bridge_hub_westend_runtime.compact.compressed.wasm on-runtime-upgrade --checks=pre-and-post live --uri wss://westend-bridge-hub-rpc.polkadot.io:443
✅ Found 6698 keys (3.23s)
[00:00:17] ✅ Downloaded key values 391.0557/s [====================================================================================================================================================] 6698/6698 (0s)
✅ Inserted keys into DB (0.03s)
[2024-09-19T01:40:40Z INFO  ethereum_system::migration] Agents and channels not initialized. Initialization will run.
[2024-09-19T01:40:40Z INFO  ethereum_system::migration] Ethereum system initialized.
[2024-09-19T01:40:40Z INFO  ethereum_system::migration] Ethereum system initialized.
[2024-09-19T01:40:40Z INFO  ethereum_system::migration] Agents and channels are initialized. Initialization will not run.
[2024-09-19T01:40:40Z INFO  ethereum_system::migration] Ethereum system already initialized. Skipping.

@acatangiu Is there anything required left for this PR?

@acatangiu
Copy link
Contributor

@yrong just some spurious network issues leading to some CI tests failing, restarted those CI jobs, should be auto-merged when they finish

@acatangiu acatangiu added this pull request to the merge queue Sep 19, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A4-needs-backport Pull request must be backported to all maintained releases. T15-bridges This PR/Issue is related to bridges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants