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

Update substrate/polkadot/cumulus from v0.9.7 to v0.9.8 #586

Closed
github-actions bot opened this issue Jul 8, 2021 · 7 comments
Closed

Update substrate/polkadot/cumulus from v0.9.7 to v0.9.8 #586

github-actions bot opened this issue Jul 8, 2021 · 7 comments

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Jul 8, 2021

Description

This ticket is automatically generated using

$ yarn run print-version-bump-info --from v0.9.7 --to v0.9.8

substrate (2021-06-18 -> 2021-06-30)

paritytech/substrate@polkadot-v0.9.7...polkadot-v0.9.8

    from: 9c572625f6557dfdb19f47474369a0327d51dfbc
      to: 1b758b2a8d151d97d2242260c465b6df9cb8a7a4
 commits: 59

polkadot (2021-06-25 -> 2021-07-06)

paritytech/polkadot@release-v0.9.7...release-v0.9.8

    from: 5d35bac7408a4cb12a578764217d06f3920b36aa
      to: 3a10ee63c0b5703a1c802db3438ab7e01344a8ce
 commits: 72

cumulus (2021-06-30 -> 2021-07-08)

paritytech/cumulus@polkadot-v0.9.7...polkadot-v0.9.8

    from: c5c3abf7eb9d4fdfb588d6560efaa8dca66a8dbc
      to: 4e51d32fdc4636f2d207204a25534e01d5fddf75
 commits: 19

Important commits by label

M5-statemint

(paritytech/cumulus#516) Propose bumping StringLimit to 128 bytes

M1-ci

(paritytech/cumulus#526) upgrade srtool-actions and subwasm

E5-breaksapi

(paritytech/substrate#9106) Frame remove_all with size limit.
(paritytech/substrate#9220) Move client only primitives to another dir

E4-newhostfunctions

(paritytech/substrate#9106) Frame remove_all with size limit.

E1-runtimemigration

(paritytech/substrate#8920) Add Control to Growth of the Staking Pallet
(paritytech/substrate#9088) Migrate pallet-elections to pallet attribute macro
(paritytech/substrate#9087) Migrate pallet-transaction-payment to new pallet attribute macro
(paritytech/polkadot#3299) Set new staking limits

D9-needsaudit 👮

(paritytech/substrate#9193) Improve Staking Limits
(paritytech/polkadot#2793) Companion for Decouple Staking and Election - Part 3: Signed Phase

D2-breaksapi

(paritytech/substrate#9106) Frame remove_all with size limit.
(paritytech/substrate#9119) Result<Option<>> rather than Option<Option<>>

C7-high ❗️

(paritytech/substrate#8920) Add Control to Growth of the Staking Pallet
(paritytech/substrate#9102) double the allocator limit
(paritytech/substrate#7910) Decouple Staking and Election - Part 3: Signed Phase
(paritytech/polkadot#3299) Set new staking limits

BD0-await-business-review

(paritytech/substrate#8623) Add PolkaFoundry, PolkaSmith SS58 address
(paritytech/substrate#9123) Prep for Altair - Add ss58 prefix

B7-runtimenoteworthy

(paritytech/substrate#9106) Frame remove_all with size limit.
(paritytech/substrate#8920) Add Control to Growth of the Staking Pallet
(paritytech/substrate#9119) Result<Option<>> rather than Option<Option<>>
(paritytech/substrate#8976) contracts: Remove weight pre charging
(paritytech/substrate#7910) Decouple Staking and Election - Part 3: Signed Phase
(paritytech/substrate#9193) Improve Staking Limits
(paritytech/polkadot#3249) Use Safe Math in XCM Builder
(paritytech/polkadot#3237) Allow Council to Use Scheduler
(paritytech/polkadot#3299) Set new staking limits
(paritytech/polkadot#3329) Use max_code_size and max_wasm_data_size from Parachains Configuration
(paritytech/polkadot#3346) Do not allow any crowdloan contributions during the VRF period
(paritytech/polkadot#3361) Don't allow bids for a ParaId where there is an overlapping lease period

B5-clientnoteworthy

(paritytech/substrate#8998) Decommit instance memory after a runtime call on Linux
(paritytech/substrate#9019) Make it possible to override maximum payload of RPC
(paritytech/substrate#9102) double the allocator limit
(paritytech/substrate#8884) Fast sync
(paritytech/substrate#9217) Remove txpool as an export of sc_transaction_pool, exporting the used components instead.
(paritytech/substrate#9220) Move client only primitives to another dir

B3-apinoteworthy

(paritytech/substrate#8949) Emit error when construct_runtime imports a non-existent pallet part

B1-releasenotes

(paritytech/polkadot#3270) improved gossip topology
(paritytech/polkadot#3333) Only fetch one collation at a time per relay parent

N/A

(paritytech/cumulus#496) WIP: Update Substrate & Polkadot
(paritytech/cumulus#497) Update Polkadot & Substrate
(paritytech/cumulus#501) Fix Max Block Weight
(paritytech/cumulus#502) Fix Benchmark Rename
(paritytech/cumulus#498) min collator check
(paritytech/cumulus#503) Fix CandidateRemoved event
(paritytech/cumulus#484) Adds a mock provider for the parachain inherent
(paritytech/cumulus#507) Refactor paths and runtime names
(paritytech/cumulus#515) add RelaychainBlockNumberProvider
(paritytech/cumulus#519) Update Substrate & Polkadot
(paritytech/cumulus#523) Update Substrate & Polkadot
(paritytech/cumulus#525) Bring back runtime upgrade test
(paritytech/cumulus#510) Fix xcmp message processing condition.

@JoshOrndorff
Copy link
Contributor

I've determined that we do not need a storage migration to reflect paritytech/substrate#9087 because the same string, TransactionPayment was used in both the old decl_storage! definition, and the construct_runtime! definition in each of the moonbase, moonbeam, moonshadow, and moonriver runtimes.

@JoshOrndorff
Copy link
Contributor

Also, the ticket includes a few things that we've already handled in previous dependency updates. Notably this one:

(paritytech/substrate#9102) double the allocator limit

@JoshOrndorff
Copy link
Contributor

Also the ticket is missing one that turned out to be quite relevant and is the reason for the broken evm tracing. paritytech/substrate#8953

@crystalin
Copy link
Collaborator

@JoshOrndorff good catch,
@librelois can you check why the script didn't catch this PR ? I suspect it is the labels being filtered :/
(you can run the script locally, you don't need to trigger the CI).

@librelois
Copy link
Collaborator

Also the ticket is missing one that turned out to be quite relevant and is the reason for the broken evm tracing. paritytech/substrate#8953

@librelois can you check why the script didn't catch this PR ? I suspect it is the labels being filtered :/
(you can run the script locally, you don't need to trigger the CI).

No, it's not a label problem. This PR contains the label B5 which would pass the filter. The problem comes from the git tree of the substrate repository, this PR is not located between the tags polkadot-v0.9.7 and polkadot-v0.9.8. The corresponding commit does not appear here: paritytech/substrate@polkadot-v0.9.7...polkadot-v0.9.8

@crystalin
Copy link
Collaborator

@librelois , It does appear there: the one before last:
image

@librelois
Copy link
Collaborator

My bad, I didn't find it with a ctrl+F (probably a mistyping). Anyway I have the same behavior when running the script locally, and this PR is not listed in the PR with the B5 tag, even before filtering by label.

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

No branches or pull requests

3 participants