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

Snowbridge - Ethereum Client - Public storage items #4504

Conversation

claravanstaden
Copy link
Contributor

This is a cherry-pick from master of #4501

Expected patches for (1.7.0):
snowbridge-pallet-ethereum-client

@paritytech-review-bot paritytech-review-bot bot requested a review from a team May 17, 2024 11:00
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6235473

@bkontur bkontur added R0-silent Changes should not be mentioned in any release notes T15-bridges This PR/Issue is related to bridges. labels May 17, 2024
@acatangiu acatangiu merged commit a3dfa12 into paritytech:release-crates-io-v1.7.0 May 17, 2024
19 of 133 checks passed
github-merge-queue bot pushed a commit that referenced this pull request May 17, 2024
Changes the Ethereum client storage scope to public, so it can be set in
a migration.

When merged, we should backport to the all other release branches:

- [ ] release-crates-io-v1.7.0 - patch release the fellows BridgeHubs
runtimes #4504
- [ ] release-crates-io-v1.8.0 -
#4505
- [ ] release-crates-io-v1.9.0 -
#4506
- [ ] release-crates-io-v1.10.0 -
#4507
- [ ] release-crates-io-v1.11.0 -
#4508
- [ ] release-crates-io-v1.12.0 (commit soon)
fellowship-merge-bot bot pushed a commit to polkadot-fellows/runtimes that referenced this pull request May 19, 2024
Upgrades Snowbridge with an Ethereum client fixes: 
- paritytech/polkadot-sdk#4504
- paritytech/polkadot-sdk#4482

Adds a migration to reset the Ethereum checkpoint. Will be reset again
to the moment recent checkpoint before merged.

TODO:
- [x] Check migration works
- [x] Check converting Sync Committee to prepared keys is OK for a
migration

---------

Co-authored-by: Adrian Catangiu <[email protected]>
Co-authored-by: Vincent Geddes <[email protected]>
Co-authored-by: ron <[email protected]>
hitchhooker pushed a commit to ibp-network/polkadot-sdk that referenced this pull request Jun 5, 2024
Changes the Ethereum client storage scope to public, so it can be set in
a migration.

When merged, we should backport to the all other release branches:

- [ ] release-crates-io-v1.7.0 - patch release the fellows BridgeHubs
runtimes paritytech#4504
- [ ] release-crates-io-v1.8.0 -
paritytech#4505
- [ ] release-crates-io-v1.9.0 -
paritytech#4506
- [ ] release-crates-io-v1.10.0 -
paritytech#4507
- [ ] release-crates-io-v1.11.0 -
paritytech#4508
- [ ] release-crates-io-v1.12.0 (commit soon)
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
Changes the Ethereum client storage scope to public, so it can be set in
a migration.

When merged, we should backport to the all other release branches:

- [ ] release-crates-io-v1.7.0 - patch release the fellows BridgeHubs
runtimes paritytech#4504
- [ ] release-crates-io-v1.8.0 -
paritytech#4505
- [ ] release-crates-io-v1.9.0 -
paritytech#4506
- [ ] release-crates-io-v1.10.0 -
paritytech#4507
- [ ] release-crates-io-v1.11.0 -
paritytech#4508
- [ ] release-crates-io-v1.12.0 (commit soon)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes T15-bridges This PR/Issue is related to bridges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants