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

chore(upgrade): v0.9.43 to v1.0.0 #1700

Merged
merged 1 commit into from
Oct 20, 2023

Conversation

mattheworris
Copy link
Collaborator

@mattheworris mattheworris commented Oct 5, 2023

Goal

The goal of this PR is to upgrade to Polkadot v1.0.0

Notable changes:

  • The docker images for the local relay network have been updated to
    v1.1.0 to accurately reflect the last working version of Rococo-testnet.

  • ci-base-image has been updated to install the stable and nightly
    toolchains.

References

Closes #1639
Closes #1605

Checklist

  • Chain spec updated
  • Design doc(s) updated
  • Weights updated

@mattheworris mattheworris linked an issue Oct 5, 2023 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Oct 10, 2023

Codecov Report

Merging #1700 (559180e) into main (098cac0) will increase coverage by 4.78%.
The diff coverage is 95.34%.

@@            Coverage Diff             @@
##             main    #1700      +/-   ##
==========================================
+ Coverage   83.09%   87.88%   +4.78%     
==========================================
  Files          47       50       +3     
  Lines        4491     4241     -250     
==========================================
- Hits         3732     3727       -5     
+ Misses        759      514     -245     
Files Coverage Δ
pallets/capacity/src/lib.rs 88.37% <100.00%> (+11.80%) ⬆️
pallets/frequency-tx-payment/src/lib.rs 84.61% <ø> (+2.06%) ⬆️
pallets/handles/src/lib.rs 92.41% <100.00%> (+6.83%) ⬆️
pallets/messages/src/lib.rs 89.40% <100.00%> (+7.08%) ⬆️
pallets/msa/src/lib.rs 90.00% <100.00%> (+6.34%) ⬆️
pallets/msa/src/types.rs 100.00% <100.00%> (ø)
pallets/stateful-storage/src/lib.rs 86.12% <100.00%> (+7.74%) ⬆️
pallets/stateful-storage/src/types.rs 90.72% <ø> (+5.14%) ⬆️
pallets/schemas/src/lib.rs 85.50% <0.00%> (+6.17%) ⬆️
pallets/time-release/src/lib.rs 92.30% <50.00%> (+9.81%) ⬆️

... and 15 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions github-actions bot added metadata-changed Metadata has changed since the latest full release metadata-version-not-incremented Metadata has changed since the latest full release, but the version has not been incremented and removed metadata-changed Metadata has changed since the latest full release metadata-version-not-incremented Metadata has changed since the latest full release, but the version has not been incremented labels Oct 10, 2023
@mattheworris mattheworris marked this pull request as ready for review October 10, 2023 16:15
@github-actions github-actions bot removed metadata-changed Metadata has changed since the latest full release metadata-version-not-incremented Metadata has changed since the latest full release, but the version has not been incremented labels Oct 10, 2023
@mattheworris mattheworris requested a review from a team October 10, 2023 16:15
@github-actions github-actions bot added metadata-changed Metadata has changed since the latest full release and removed metadata-changed Metadata has changed since the latest full release labels Oct 10, 2023
@github-actions
Copy link

Running benchmarks and calculating weights. DO NOT MERGE! A new commit will be added upon completion...

@github-actions github-actions bot added the metadata-changed Metadata has changed since the latest full release label Oct 10, 2023
@github-actions
Copy link

Running benchmarks and calculating weights. DO NOT MERGE! A new commit will be added upon completion...

@github-actions
Copy link

Finished running benchmarks. Updated weights have been committed to this PR branch in commit 0270497.

demisx pushed a commit that referenced this pull request Oct 10, 2023
@github-actions github-actions bot added metadata-changed Metadata has changed since the latest full release and removed metadata-changed Metadata has changed since the latest full release labels Oct 10, 2023
@JoeCap08055
Copy link
Collaborator

lgtm, but I won't approve just yet so others have time to review; also, I maybe don't have as much context for some of the changes. lmk if this hangs out & you need an approval, though.

Also, thank you Rust for making it so we need 2 different toolchain versions 😐

@enddynayn enddynayn force-pushed the 1639-upgrade-to-polkadot-release-v100 branch from 1078773 to 80e507e Compare October 19, 2023 00:25
@github-actions github-actions bot added metadata-changed Metadata has changed since the latest full release and removed metadata-changed Metadata has changed since the latest full release labels Oct 19, 2023
@github-actions
Copy link

Running benchmarks and calculating weights. DO NOT MERGE! A new commit will be added upon completion...

@github-actions
Copy link

Finished running benchmarks. Updated weights have been committed to this PR branch in commit 16a1c4b.

demisx pushed a commit that referenced this pull request Oct 19, 2023
@github-actions github-actions bot added the metadata-changed Metadata has changed since the latest full release label Oct 19, 2023
@enddynayn enddynayn force-pushed the 1639-upgrade-to-polkadot-release-v100 branch from 16a1c4b to adff621 Compare October 19, 2023 18:10
@github-actions github-actions bot removed the metadata-changed Metadata has changed since the latest full release label Oct 19, 2023
@enddynayn enddynayn marked this pull request as ready for review October 19, 2023 19:34
@github-actions github-actions bot added the metadata-changed Metadata has changed since the latest full release label Oct 19, 2023
Copy link
Collaborator

@shannonwells shannonwells left a comment

Choose a reason for hiding this comment

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

🚢 🦑

Copy link
Collaborator

@JoeCap08055 JoeCap08055 left a comment

Choose a reason for hiding this comment

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

:shipit:

Copy link
Collaborator

@saraswatpuneet saraswatpuneet left a comment

Choose a reason for hiding this comment

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

@github-actions github-actions bot removed the metadata-changed Metadata has changed since the latest full release label Oct 20, 2023
@mattheworris mattheworris force-pushed the 1639-upgrade-to-polkadot-release-v100 branch from f1ec2bc to 19ef6a7 Compare October 20, 2023 18:31
@mattheworris mattheworris force-pushed the 1639-upgrade-to-polkadot-release-v100 branch from 19ef6a7 to 76429e1 Compare October 20, 2023 18:45
@mattheworris mattheworris force-pushed the 1639-upgrade-to-polkadot-release-v100 branch 2 times, most recently from 459f414 to 2fb03c6 Compare October 20, 2023 20:51
@mattheworris mattheworris enabled auto-merge (squash) October 20, 2023 21:08
The goal of this PR is to upgrade to Polkadot v1.0.0

Notable changes:

- The docker images for the local relay network have been updated to
  v1.1.0 to accurately reflect the last working version of Rococo-testnet.

- ci-base-image has been updated to install the stable and nightly
  toolchains.

References
- [Polkadot Forum Release Notes](https://forum.polkadot.network/t/polkadot-release-analysis-v1-0-0/3585)
- [V1.0.0 Release Tag](https://github.com/paritytech/polkadot/releases/tag/v1.0.0)
- [Polkadot-v0.9.43 vs Polkadot-v1.0.0 difference](paritytech/substrate@polkadot-v0.9.43...polkadot-v1.0.0#diff-e5be9978262555d1f400caa5ae803ffc96be0ba25fd4ed8bd917c12529474c44R105)

issue-1639
issue-1605
@mattheworris mattheworris force-pushed the 1639-upgrade-to-polkadot-release-v100 branch from a3a2add to 559180e Compare October 20, 2023 21:13
@github-actions github-actions bot added the metadata-changed Metadata has changed since the latest full release label Oct 20, 2023
@mattheworris mattheworris merged commit b12638d into main Oct 20, 2023
30 checks passed
@mattheworris mattheworris deleted the 1639-upgrade-to-polkadot-release-v100 branch October 20, 2023 22:06
mattheworris pushed a commit that referenced this pull request Oct 24, 2023
This PR aims to remove the `rustup install toolchain nightly` commands from the individual
GitHub jobs that use the `ci-base-image` that previously did not have the `nightly` toolchain.

Closes #1727

 # Discussion
The changes in the previous PR (#1700) to add the `nightly` toolchain to `ci-base-image` were
not published to DockerHub until the `merge-pr.yml` workflow actions were successfully run.
Therefore the `nightly` toolchain was not available for CI to use in `ci-base-image`.

In that PR, the `nightly` toolchain was installed in the individual jobs where it was needed to pass CI.

Now that the PR has been merged, `ci-base-image` has been updated with both the `stable`
and `nightly` toolchains and the redundant `rustup install` commands can be removed from `verify-pr-commit.yml`.
mattheworris added a commit that referenced this pull request Oct 25, 2023
…ci-base-image update (#1728)

# Goal
This PR aims to remove the `rustup install toolchain nightly` commands
from the individual GitHub jobs that use the `ci-base-image` that
previously did not have the `nightly` toolchain.

Closes #1727

# Discussion
The changes in the previous PR (#1700) to add the `nightly` toolchain to
`ci-base-image` were not published to DockerHub until the `merge-pr.yml`
workflow actions were successfully run. Therefore the `nightly`
toolchain was not available for CI to use in `ci-base-image`.

In that PR, the `nightly` toolchain was installed in the individual jobs
where it was needed to pass CI.

Now that the PR has been merged, `ci-base-image` has been updated with
both the `stable` and `nightly` toolchains and the redundant `rustup
install` commands can be removed from `verify-pr-commit.yml`.

Co-authored-by: Matthew Orris <--help>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
metadata-changed Metadata has changed since the latest full release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to Polkadot Release v1.0.0 Support stable rust for compiling the runtime
8 participants