Make CI succeed and add migrations #1329

sea212 · 2024-04-30T14:54:55Z

What does it do?

Make CI pass
Add migrations

What important points should reviewers know?

pallet-contracts required pallet-balance hold and freezes
Current PalletVersion of pallet-contracts is at 9, which indicates the storage layout version.

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues?

References

maltekliemann

Approved minus the zeitgeist/battery-station typo

mergify · 2024-04-30T16:36:43Z

This pull request is now in conflicts. Could you fix it @sea212? 🙏

…ependencies-part-7

codecov-commenter · 2024-04-30T21:40:39Z

Codecov Report

Attention: Patch coverage is 90.00000% with 2 lines in your changes are missing coverage. Please review.

❗ No coverage uploaded for pull request base (sea212-update-dependencies-v1.1.0@7fc38ee). Click here to learn what that means.

Files	Patch %	Lines
zrml/asset-router/src/pallet_impl/unbalanced.rs	33.33%	2 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@                         Coverage Diff                          @@
##             sea212-update-dependencies-v1.1.0    #1329   +/-   ##
====================================================================
  Coverage                                     ?   92.52%           
====================================================================
  Files                                        ?      198           
  Lines                                        ?    36036           
  Branches                                     ?        0           
====================================================================
  Hits                                         ?    33343           
  Misses                                       ?     2693           
  Partials                                     ?        0

Flag	Coverage Δ
tests	`92.52% <90.00%> (?)`

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Chralt98

I got this as a blue print:

#1079

From this I derive to look for the following repos:

substrate
cumulus
moonbeam
nimbus
orml
polkadot

I found the following migrations:

Using paritytech/cumulus@polkadot-v0.9.38...polkadot-v1.0.0

pallets/dmp-queue/src/migration.rs
pallets/parachain-system/src/migration.rs
pallets/xcmp-queue/src/migration.rs

I need to check if these migrations are automatically triggered by the on_runtime_hook. If yes, we can leave them out (because of implicit trigger) and don't have to state them explicitly in our migrations. So, OnRuntimeUpgrade is implemented. So I derive it's automatically triggered by our runtime executor.

Did we already execute this migration? https://github.com/paritytech/polkadot-sdk/blob/release-polkadot-v1.1.0/substrate/frame/contracts/src/migration/v09.rs

Using moonbeam-foundation/moonbeam@v0.31.1...v0.34.1

I found no migrations for that.

https://github.com/moonbeam-foundation/nimbus

The last version of nimbus is polkadot-v0.9.43 So now the question arises what are we going to do about this problem?

I found no migration for open-web3-stack/open-runtime-module-library@polkadot-v0.9.38...polkadot-v1.1.0

paritytech/polkadot@release-v0.9.38...release-v1.0.0

runtime/common/src/session/migration.rs
Those are executed implicitly runtime/parachains/src/configuration/migration/v5.rs and this runtime/parachains/src/configuration/migration/v6.rs and this runtime/parachains/src/configuration/migration_ump.rs , right?

paritytech/substrate@polkadot-v0.9.38...polkadot-v1.0.0

frame/contracts/src/migration.rs

frame/democracy/src/migrations/mod.rs (automatically triggered through OnRuntimeUpgrade hook)

Those are the commits from the monorepo transition to 1.1.0:

For cumulus:
paritytech/polkadot-sdk@af7ae72...polkadot-v1.1.0

For polkadot:
paritytech/polkadot-sdk@7c89f28...polkadot-v1.1.0

substrate/frame/bounties/src/migrations/v4.rs

substrate/frame/membership/src/migrations/v4.rs

Please check and verify if my information provided is correct.

Chralt98 · 2024-05-06T07:47:33Z

runtime/battery-station/src/parameters.rs

+    pub const MaxHolds: u32 = 1;
+    pub const MaxFreezes: u32 = 1;


Are we sure about these values?

Chralt98 · 2024-05-06T07:48:16Z

runtime/zeitgeist/src/parameters.rs

+    pub const MaxHolds: u32 = 1;
+    pub const MaxFreezes: u32 = 1;


Are we sure about these values?

Seems fine to me. I don't think we use freezes or holds yet, so... If you have a suggestion though, let's hear it.

I just thought about setting the values to zero instead, because it was like this before. But maybe Harald had a particular reason for using one for both.

maltekliemann · 2024-05-06T14:39:04Z

I got this as a blue print:

#1079

From this I derive to look for the following repos:

substrate

cumulus

moonbeam

nimbus

orml

polkadot

I found the following migrations:

Using paritytech/[email protected]

pallets/dmp-queue/src/migration.rs pallets/parachain-system/src/migration.rs pallets/xcmp-queue/src/migration.rs

I need to check if these migrations are automatically triggered by the on_runtime_hook. If yes, we can leave them out (because of implicit trigger) and don't have to state them explicitly in our migrations. So, OnRuntimeUpgrade is implemented. So I derive it's automatically triggered by our runtime executor.

These should have already been executed on our chain. The migrations have been added before and only been changed in the commit range you've checked (due to import reorgs or clippy requests). You can also compare the on-chain version with what's in the code.

Did we already execute this migration? https://github.com/paritytech/polkadot-sdk/blob/release-polkadot-v1.1.0/substrate/frame/contracts/src/migration/v09.rs

Yes, pallet-contracts v09 predates 0.9.38; our contracts pallet is at storage version 9.

https://github.com/moonbeam-foundation/nimbus

The last version of nimbus is polkadot-v0.9.43 So now the question arises what are we going to do about this problem?

You should be able to find what you're looking for in https://github.com/zeitgeistpm/moonkit.

I found no migration for open-web3-stack/[email protected]

paritytech/[email protected]

runtime/common/src/session/migration.rs Those are executed implicitly runtime/parachains/src/configuration/migration/v5.rs and this runtime/parachains/src/configuration/migration/v6.rs and this runtime/parachains/src/configuration/migration_ump.rs , right?

Doesn't matter as these are all relaychain specific packages.

paritytech/[email protected]

frame/contracts/src/migration.rs

frame/democracy/src/migrations/mod.rs (automatically triggered through OnRuntimeUpgrade hook)

The contracts migration should be covered by this PR, while the changes to democracy are already deployed (the commits only seem to move some stuff around, not add a migration).

Those are the commits from the monorepo transition to 1.1.0:

For cumulus: paritytech/[email protected]

For polkadot: paritytech/[email protected]

substrate/frame/bounties/src/migrations/v4.rs

As far as I understand, we're already at the correct storage state, it's just that Parity failed to upgrade StorageVersion, which is why we're behind.

substrate/frame/membership/src/migrations/v4.rs

Also, the link that you've provided doesn't show a new migration, as far as I can see. Only the copyright has been updated.

Please check and verify if my information provided is correct.

Appreciate the thorough review!

Chralt98

#1329 (comment)

Please just look at the comment above before.

Otherwise LGTM

* Update dependencies (#1319) * Update dependencies to polkadot-v1.1.0 * Format code * Remove duplicate dependencies * Update zrml-asset-router (#1321) * Update zrml-primitives * Partially update asset-router * Finalize logic adjustments in asset-router * Make asset-router tests compilable * Correct Inspect routing for market assets in Currencies * Directly invoke Inspect API for Currencies * Add tests for remaining Unbalances functions * Update remaining Zeitgeist pallets (#1322) * Update zrml-asset-router (#1321) * Upgrade zrml-market-commons * Upgrade zrml-authorized && use MockBlock instead of MockBlockU32 * Upgrade zrml-court * Upgrade zrml-global-disputes * Upgrade liquidity mining * Upgrade zrml-rikiddo * Upgrade zrml-simple-disputes * Upgrade zrml-styx * Upgrade zrml-orderbook * Upgrade zrml-parimutuel * Upgrade zrml-swaps * Upgrade zrml-prediction-markets * Upgrade zrml-neo-swaps * Upgrade zrml-hybrid-router * Update license headers * Update runtime (#1323) * Update weight files & Runtime enum * Use workspace metadata * Always use serde serialization for asset types * Make battery station standalone runtime compilable * Make benchmark and try-runtime feature compilable * Make BS build with all features * Make parachain tests compile * Partially fix xcm tests * Use safe xcm version 2 * Update Zeitgeist runtime (except xcm tests) * Format code * Remove deprecated comment * Integrate new xcm-emulator (#1324) * Integrate new xcm-emulator environment * Utilize new xcm-emulator interfaces * Spawn relay-para network using patched xcm-emulator * Use proper collator genesis config * Fix Rococo tests * Finalize Battery Station XCM tests * Finalize Zeitgeist XCM tests * Update client (#1327) * Fix rpc and work on client update * Finalize standalone client * Update parachain client * Use same try-runtime subcommand in every case * Update node/src/cli.rs Co-authored-by: Malte Kliemann <[email protected]> * Update try-runtime* Makefile targets --------- Co-authored-by: Malte Kliemann <[email protected]> * Make CI succeed and add migrations (#1329) * Fix rpc and work on client update * Finalize standalone client * Update parachain client * Use same try-runtime subcommand in every case * Satisfy Clippy * Fix benchmarks * Add migrations * Satisfy Clippy * Update moonkit depedencies * Free disk space more aggressively --------- Co-authored-by: Malte Kliemann <[email protected]> * Update spec version, try-runtime Makefile * Fix copyright notices * Fix broken chain state (#1336) * Add `StorageVersion` fix and contrats fix migrations * Don't set pallet-balances' storage version * Remove migrations from pallet-contracts config * Clear storage tries of contracts * Fix migration and info logs in try-runtime * Fix licenses and comments * Fix formatting --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Update dependencies to Polkadot v1.1.0 (#1331) * Update dependencies (#1319) * Update dependencies to polkadot-v1.1.0 * Format code * Remove duplicate dependencies * Update zrml-asset-router (#1321) * Update zrml-primitives * Partially update asset-router * Finalize logic adjustments in asset-router * Make asset-router tests compilable * Correct Inspect routing for market assets in Currencies * Directly invoke Inspect API for Currencies * Add tests for remaining Unbalances functions * Update remaining Zeitgeist pallets (#1322) * Update zrml-asset-router (#1321) * Upgrade zrml-market-commons * Upgrade zrml-authorized && use MockBlock instead of MockBlockU32 * Upgrade zrml-court * Upgrade zrml-global-disputes * Upgrade liquidity mining * Upgrade zrml-rikiddo * Upgrade zrml-simple-disputes * Upgrade zrml-styx * Upgrade zrml-orderbook * Upgrade zrml-parimutuel * Upgrade zrml-swaps * Upgrade zrml-prediction-markets * Upgrade zrml-neo-swaps * Upgrade zrml-hybrid-router * Update license headers * Update runtime (#1323) * Update weight files & Runtime enum * Use workspace metadata * Always use serde serialization for asset types * Make battery station standalone runtime compilable * Make benchmark and try-runtime feature compilable * Make BS build with all features * Make parachain tests compile * Partially fix xcm tests * Use safe xcm version 2 * Update Zeitgeist runtime (except xcm tests) * Format code * Remove deprecated comment * Integrate new xcm-emulator (#1324) * Integrate new xcm-emulator environment * Utilize new xcm-emulator interfaces * Spawn relay-para network using patched xcm-emulator * Use proper collator genesis config * Fix Rococo tests * Finalize Battery Station XCM tests * Finalize Zeitgeist XCM tests * Update client (#1327) * Fix rpc and work on client update * Finalize standalone client * Update parachain client * Use same try-runtime subcommand in every case * Update node/src/cli.rs Co-authored-by: Malte Kliemann <[email protected]> * Update try-runtime* Makefile targets --------- Co-authored-by: Malte Kliemann <[email protected]> * Make CI succeed and add migrations (#1329) * Fix rpc and work on client update * Finalize standalone client * Update parachain client * Use same try-runtime subcommand in every case * Satisfy Clippy * Fix benchmarks * Add migrations * Satisfy Clippy * Update moonkit depedencies * Free disk space more aggressively --------- Co-authored-by: Malte Kliemann <[email protected]> * Update spec version, try-runtime Makefile * Fix copyright notices * Fix broken chain state (#1336) * Add `StorageVersion` fix and contrats fix migrations * Don't set pallet-balances' storage version * Remove migrations from pallet-contracts config * Clear storage tries of contracts * Fix migration and info logs in try-runtime * Fix licenses and comments * Fix formatting --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Merge * Fix benchmark * Fix compiler error * Fix tests and imports * Fix imports (again...) * Fix orderbook benchmarks * Fix fuzz tests * Fix formatting * Fix orderbook fuzz --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Revert "New Asset System (#1295)" (#1338) * Revert "New Asset System (#1295)" This reverts commit a956877. * Fix formatting * Update copyright * Remove pallet-assets dependency * Fix fuzz tests * Merge `main` into `develop` (#1345) * Update dependencies to Polkadot v1.1.0 (#1331) * Update dependencies (#1319) * Update dependencies to polkadot-v1.1.0 * Format code * Remove duplicate dependencies * Update zrml-asset-router (#1321) * Update zrml-primitives * Partially update asset-router * Finalize logic adjustments in asset-router * Make asset-router tests compilable * Correct Inspect routing for market assets in Currencies * Directly invoke Inspect API for Currencies * Add tests for remaining Unbalances functions * Update remaining Zeitgeist pallets (#1322) * Update zrml-asset-router (#1321) * Upgrade zrml-market-commons * Upgrade zrml-authorized && use MockBlock instead of MockBlockU32 * Upgrade zrml-court * Upgrade zrml-global-disputes * Upgrade liquidity mining * Upgrade zrml-rikiddo * Upgrade zrml-simple-disputes * Upgrade zrml-styx * Upgrade zrml-orderbook * Upgrade zrml-parimutuel * Upgrade zrml-swaps * Upgrade zrml-prediction-markets * Upgrade zrml-neo-swaps * Upgrade zrml-hybrid-router * Update license headers * Update runtime (#1323) * Update weight files & Runtime enum * Use workspace metadata * Always use serde serialization for asset types * Make battery station standalone runtime compilable * Make benchmark and try-runtime feature compilable * Make BS build with all features * Make parachain tests compile * Partially fix xcm tests * Use safe xcm version 2 * Update Zeitgeist runtime (except xcm tests) * Format code * Remove deprecated comment * Integrate new xcm-emulator (#1324) * Integrate new xcm-emulator environment * Utilize new xcm-emulator interfaces * Spawn relay-para network using patched xcm-emulator * Use proper collator genesis config * Fix Rococo tests * Finalize Battery Station XCM tests * Finalize Zeitgeist XCM tests * Update client (#1327) * Fix rpc and work on client update * Finalize standalone client * Update parachain client * Use same try-runtime subcommand in every case * Update node/src/cli.rs Co-authored-by: Malte Kliemann <[email protected]> * Update try-runtime* Makefile targets --------- Co-authored-by: Malte Kliemann <[email protected]> * Make CI succeed and add migrations (#1329) * Fix rpc and work on client update * Finalize standalone client * Update parachain client * Use same try-runtime subcommand in every case * Satisfy Clippy * Fix benchmarks * Add migrations * Satisfy Clippy * Update moonkit depedencies * Free disk space more aggressively --------- Co-authored-by: Malte Kliemann <[email protected]> * Update spec version, try-runtime Makefile * Fix copyright notices * Fix broken chain state (#1336) * Add `StorageVersion` fix and contrats fix migrations * Don't set pallet-balances' storage version * Remove migrations from pallet-contracts config * Clear storage tries of contracts * Fix migration and info logs in try-runtime * Fix licenses and comments * Fix formatting --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Merge * Fix benchmark * Fix compiler error * Fix tests and imports * Fix imports (again...) * Fix orderbook benchmarks * Fix fuzz tests * Fix formatting * Fix orderbook fuzz --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Update versions to v0.5.3 * Fix Makefile * Change endpoint of integration tests * Fix copyright notices * Run benchmarks --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> Co-authored-by: Your Name <[email protected]>

* Remove structs mentioned under TODO(#986) (#1342) * Remove structs under TODO(#968) * Remove OldOutcomeInfo and related structs * Satisfy check-license * Include buy/sell hybrid-router calls in proxy filters (#1343) * Include buy/sell calls in proxy filters * Add hybrid-router calls to HandleAssets --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Remove zrml-liquidity-mining (#1341) * Remove zrml-liquidity-mining * Fix formatting * Fix copyright notices * Remove zrml-liquidity-mining from benchmarks script --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Update style guide (#1339) * Be more strict with feature branch size * Clarify our workflow * Fix formatting --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Revert "New Asset System (#1295)" (#1338) * Revert "New Asset System (#1295)" This reverts commit a956877. * Fix formatting * Update copyright * Remove pallet-assets dependency * Fix fuzz tests * Remove zrml-rikiddo (#1340) * Remove zrml-rikiddo * Remove zrml-rikiddo from README * Remove remnants of zrml-rikiddo * Add note to changelog * Remove Rikiddo constants from fuzz script * Remove code owners --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Remove zrml-simple-disputes and migrate markets (#1344) * Remove zrml-simple-disputes and migrate markets * Fix formatting * Minor improvement * Fix redundant match guard * Remove unused pallet constant * Fix formatting * Merge `main` into `develop` (#1345) * Update dependencies to Polkadot v1.1.0 (#1331) * Update dependencies (#1319) * Update dependencies to polkadot-v1.1.0 * Format code * Remove duplicate dependencies * Update zrml-asset-router (#1321) * Update zrml-primitives * Partially update asset-router * Finalize logic adjustments in asset-router * Make asset-router tests compilable * Correct Inspect routing for market assets in Currencies * Directly invoke Inspect API for Currencies * Add tests for remaining Unbalances functions * Update remaining Zeitgeist pallets (#1322) * Update zrml-asset-router (#1321) * Upgrade zrml-market-commons * Upgrade zrml-authorized && use MockBlock instead of MockBlockU32 * Upgrade zrml-court * Upgrade zrml-global-disputes * Upgrade liquidity mining * Upgrade zrml-rikiddo * Upgrade zrml-simple-disputes * Upgrade zrml-styx * Upgrade zrml-orderbook * Upgrade zrml-parimutuel * Upgrade zrml-swaps * Upgrade zrml-prediction-markets * Upgrade zrml-neo-swaps * Upgrade zrml-hybrid-router * Update license headers * Update runtime (#1323) * Update weight files & Runtime enum * Use workspace metadata * Always use serde serialization for asset types * Make battery station standalone runtime compilable * Make benchmark and try-runtime feature compilable * Make BS build with all features * Make parachain tests compile * Partially fix xcm tests * Use safe xcm version 2 * Update Zeitgeist runtime (except xcm tests) * Format code * Remove deprecated comment * Integrate new xcm-emulator (#1324) * Integrate new xcm-emulator environment * Utilize new xcm-emulator interfaces * Spawn relay-para network using patched xcm-emulator * Use proper collator genesis config * Fix Rococo tests * Finalize Battery Station XCM tests * Finalize Zeitgeist XCM tests * Update client (#1327) * Fix rpc and work on client update * Finalize standalone client * Update parachain client * Use same try-runtime subcommand in every case * Update node/src/cli.rs Co-authored-by: Malte Kliemann <[email protected]> * Update try-runtime* Makefile targets --------- Co-authored-by: Malte Kliemann <[email protected]> * Make CI succeed and add migrations (#1329) * Fix rpc and work on client update * Finalize standalone client * Update parachain client * Use same try-runtime subcommand in every case * Satisfy Clippy * Fix benchmarks * Add migrations * Satisfy Clippy * Update moonkit depedencies * Free disk space more aggressively --------- Co-authored-by: Malte Kliemann <[email protected]> * Update spec version, try-runtime Makefile * Fix copyright notices * Fix broken chain state (#1336) * Add `StorageVersion` fix and contrats fix migrations * Don't set pallet-balances' storage version * Remove migrations from pallet-contracts config * Clear storage tries of contracts * Fix migration and info logs in try-runtime * Fix licenses and comments * Fix formatting --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Merge * Fix benchmark * Fix compiler error * Fix tests and imports * Fix imports (again...) * Fix orderbook benchmarks * Fix fuzz tests * Fix formatting * Fix orderbook fuzz --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Merge develop into develop-v0.5.4 * Update versions to v0.5.3 --------- Co-authored-by: Nikhil Saboo <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> Co-authored-by: Harald Heckmann <[email protected]>

* Revert "New Asset System (#1295)" (#1338) * Revert "New Asset System (#1295)" This reverts commit a956877. * Fix formatting * Update copyright * Remove pallet-assets dependency * Fix fuzz tests * Merge `main` into `develop` (#1345) * Update dependencies to Polkadot v1.1.0 (#1331) * Update dependencies (#1319) * Update dependencies to polkadot-v1.1.0 * Format code * Remove duplicate dependencies * Update zrml-asset-router (#1321) * Update zrml-primitives * Partially update asset-router * Finalize logic adjustments in asset-router * Make asset-router tests compilable * Correct Inspect routing for market assets in Currencies * Directly invoke Inspect API for Currencies * Add tests for remaining Unbalances functions * Update remaining Zeitgeist pallets (#1322) * Update zrml-asset-router (#1321) * Upgrade zrml-market-commons * Upgrade zrml-authorized && use MockBlock instead of MockBlockU32 * Upgrade zrml-court * Upgrade zrml-global-disputes * Upgrade liquidity mining * Upgrade zrml-rikiddo * Upgrade zrml-simple-disputes * Upgrade zrml-styx * Upgrade zrml-orderbook * Upgrade zrml-parimutuel * Upgrade zrml-swaps * Upgrade zrml-prediction-markets * Upgrade zrml-neo-swaps * Upgrade zrml-hybrid-router * Update license headers * Update runtime (#1323) * Update weight files & Runtime enum * Use workspace metadata * Always use serde serialization for asset types * Make battery station standalone runtime compilable * Make benchmark and try-runtime feature compilable * Make BS build with all features * Make parachain tests compile * Partially fix xcm tests * Use safe xcm version 2 * Update Zeitgeist runtime (except xcm tests) * Format code * Remove deprecated comment * Integrate new xcm-emulator (#1324) * Integrate new xcm-emulator environment * Utilize new xcm-emulator interfaces * Spawn relay-para network using patched xcm-emulator * Use proper collator genesis config * Fix Rococo tests * Finalize Battery Station XCM tests * Finalize Zeitgeist XCM tests * Update client (#1327) * Fix rpc and work on client update * Finalize standalone client * Update parachain client * Use same try-runtime subcommand in every case * Update node/src/cli.rs Co-authored-by: Malte Kliemann <[email protected]> * Update try-runtime* Makefile targets --------- Co-authored-by: Malte Kliemann <[email protected]> * Make CI succeed and add migrations (#1329) * Fix rpc and work on client update * Finalize standalone client * Update parachain client * Use same try-runtime subcommand in every case * Satisfy Clippy * Fix benchmarks * Add migrations * Satisfy Clippy * Update moonkit depedencies * Free disk space more aggressively --------- Co-authored-by: Malte Kliemann <[email protected]> * Update spec version, try-runtime Makefile * Fix copyright notices * Fix broken chain state (#1336) * Add `StorageVersion` fix and contrats fix migrations * Don't set pallet-balances' storage version * Remove migrations from pallet-contracts config * Clear storage tries of contracts * Fix migration and info logs in try-runtime * Fix licenses and comments * Fix formatting --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Merge * Fix benchmark * Fix compiler error * Fix tests and imports * Fix imports (again...) * Fix orderbook benchmarks * Fix fuzz tests * Fix formatting * Fix orderbook fuzz --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> * Update versions to v0.5.3 * Restructure math module * More scaffolding * Implement combinatorial buy math * Implement price calculation for combo * Remove `println!` * Implement equalization * Implement selling * Add tests for combinatorial buying * Add more tests for combinatorial buying * Add tests for equalization * Add more tests/corner cases * Implement full testing, fix critical bug --------- Co-authored-by: Harald Heckmann <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

sea212 added 5 commits April 29, 2024 17:39

Fix rpc and work on client update

69fd390

Finalize standalone client

e58f4cf

Update parachain client

6f94464

Use same try-runtime subcommand in every case

4b766bb

Satisfy Clippy

b958393

sea212 added the s:in-progress The pull requests is currently being worked on label Apr 30, 2024

sea212 self-assigned this Apr 30, 2024

maltekliemann approved these changes Apr 30, 2024

View reviewed changes

mergify bot added s:revision-needed The pull requests must be revised and removed s:in-progress The pull requests is currently being worked on labels Apr 30, 2024

Merge branch 'sea212-update-dependencies-v1.1.0' into sea212-update-d…

380f811

…ependencies-part-7

mergify bot added s:in-progress The pull requests is currently being worked on and removed s:revision-needed The pull requests must be revised labels Apr 30, 2024

sea212 added 2 commits April 30, 2024 21:37

Fix benchmarks

9d6c870

Add migrations

8f5f171

sea212 added s:review-needed The pull request requires reviews and removed s:in-progress The pull requests is currently being worked on labels Apr 30, 2024

sea212 marked this pull request as ready for review April 30, 2024 21:48

sea212 requested a review from Chralt98 as a code owner April 30, 2024 21:48

Satisfy Clippy

e196b75

mergify bot added s:in-progress The pull requests is currently being worked on and removed s:review-needed The pull request requires reviews labels Apr 30, 2024

sea212 added s:review-needed The pull request requires reviews and removed s:in-progress The pull requests is currently being worked on labels Apr 30, 2024

Update moonkit depedencies

285187b

mergify bot added s:in-progress The pull requests is currently being worked on and removed s:review-needed The pull request requires reviews labels May 1, 2024

sea212 removed the s:in-progress The pull requests is currently being worked on label May 1, 2024

sea212 added the s:review-needed The pull request requires reviews label May 1, 2024

Free disk space more aggressively

332c6a3

maltekliemann self-assigned this May 5, 2024

mergify bot added s:in-progress The pull requests is currently being worked on and removed s:review-needed The pull request requires reviews labels May 5, 2024

maltekliemann added s:review-needed The pull request requires reviews and removed s:in-progress The pull requests is currently being worked on labels May 5, 2024

Chralt98 reviewed May 6, 2024

View reviewed changes

Chralt98 approved these changes May 7, 2024

View reviewed changes

maltekliemann added s:accepted This pull request is ready for merge and removed s:review-needed The pull request requires reviews labels May 7, 2024

mergify bot merged commit 718ff99 into sea212-update-dependencies-v1.1.0 May 7, 2024
31 checks passed

mergify bot deleted the sea212-update-dependencies-part-7 branch May 7, 2024 10:44

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Make CI succeed and add migrations #1329

Make CI succeed and add migrations #1329

sea212 commented Apr 30, 2024 •

edited

Loading

maltekliemann left a comment

mergify bot commented Apr 30, 2024

codecov-commenter commented Apr 30, 2024 •

edited

Loading

Chralt98 left a comment

Chralt98 May 6, 2024

Chralt98 May 6, 2024

maltekliemann May 6, 2024

Chralt98 May 7, 2024

maltekliemann commented May 6, 2024

Chralt98 left a comment

Make CI succeed and add migrations #1329

Make CI succeed and add migrations #1329

Conversation

sea212 commented Apr 30, 2024 • edited Loading

What does it do?

What important points should reviewers know?

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues?

References

maltekliemann left a comment

Choose a reason for hiding this comment

mergify bot commented Apr 30, 2024

codecov-commenter commented Apr 30, 2024 • edited Loading

Codecov Report

Chralt98 left a comment

Choose a reason for hiding this comment

Chralt98 May 6, 2024

Choose a reason for hiding this comment

Chralt98 May 6, 2024

Choose a reason for hiding this comment

maltekliemann May 6, 2024

Choose a reason for hiding this comment

Chralt98 May 7, 2024

Choose a reason for hiding this comment

maltekliemann commented May 6, 2024

Chralt98 left a comment

Choose a reason for hiding this comment

sea212 commented Apr 30, 2024 •

edited

Loading

codecov-commenter commented Apr 30, 2024 •

edited

Loading