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

split runtimes #420

Merged
merged 10 commits into from
Nov 4, 2021
Merged

split runtimes #420

merged 10 commits into from
Nov 4, 2021

Conversation

ETeissonniere
Copy link
Member

@ETeissonniere ETeissonniere commented Oct 29, 2021

In preparation of our staking testnet and parachain readyness efforts, this PR is breaking our current runtime in two different runtimes: one being used on our main and test networks and one specialized on staking. In the future, we will likely have a shell runtime used for when we start the parachain.

The support at the node level for the runtime_staking will be done in a separate PR. There is an issue for it in #422

@codecov
Copy link

codecov bot commented Oct 29, 2021

Codecov Report

Merging #420 (36f5e5a) into master (998fe08) will decrease coverage by 1.36%.
The diff coverage is 9.84%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #420      +/-   ##
==========================================
- Coverage   83.18%   81.82%   -1.37%     
==========================================
  Files          59       65       +6     
  Lines        8659     8818     +159     
==========================================
+ Hits         7203     7215      +12     
- Misses       1456     1603     +147     
Impacted Files Coverage Δ
node/src/chain_spec.rs 100.00% <ø> (ø)
node/src/command.rs 0.00% <ø> (ø)
node/src/rpc.rs 0.00% <ø> (ø)
node/src/service.rs 0.00% <ø> (ø)
pallets/allocations/src/lib.rs 82.97% <ø> (ø)
pallets/reserve/src/lib.rs 59.57% <ø> (ø)
pallets/staking/src/benchmarking.rs 93.72% <ø> (ø)
pallets/staking/src/hooks.rs 76.08% <ø> (ø)
pallets/staking/src/lib.rs 83.20% <ø> (ø)
pallets/staking/src/migrations.rs 52.91% <ø> (ø)
... and 23 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 998fe08...36f5e5a. Read the comment docs.

@ETeissonniere ETeissonniere marked this pull request as ready for review November 3, 2021 11:53
@ETeissonniere ETeissonniere merged commit 661a9fe into master Nov 4, 2021
@ETeissonniere ETeissonniere deleted the eliott/split-runtimes branch November 4, 2021 10:06
ETeissonniere added a commit that referenced this pull request Jan 20, 2022
* make the allocate extrinsic free for the oracles (#418)

* make the allocate extrinsic free for the oracles

* bump version numbers

* add staking-local spec (#419)

* update decimals (#421)

* split runtimes (#420)

* move and rename runtime to one subfolder

* remove unused deps on pallet-staking

* remove old migrations code

* create minimum staking runtime

* remove with-staking feature flag

* remove unused spec

* remove executor crate

* remove nodle_ and nodle_chain_ prefixes

* make node work with runtime_main

* fix benchmarking

* fix srtool config (#426)

* fix folder

* fix package name

* freeze rust version

* rename artifact

* fix github action issues

* master only

* bump version for release

* upgrade to polkadot-v0.9.12 (#424)

* upgrade to polkadot-v0.9.12

Signed-off-by: R.RajeshKumar <[email protected]>

* review comments fix

Signed-off-by: R.RajeshKumar <[email protected]>

* fmt fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow bringup

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow bringup

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow bringup

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow bringup

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow bringup

Signed-off-by: R.RajeshKumar <[email protected]>

* Review changes

* Staking migration updates
* Addition of rust-toolchain.toml

* ci workflow bringup

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* ci workflow fix

Signed-off-by: R.RajeshKumar <[email protected]>

* Bump serde_json from 1.0.69 to 1.0.70 (#428)

Bumps [serde_json](https://github.com/serde-rs/json) from 1.0.69 to 1.0.70.
- [Release notes](https://github.com/serde-rs/json/releases)
- [Commits](serde-rs/json@v1.0.69...v1.0.70)

---
updated-dependencies:
- dependency-name: serde_json
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump serde_json from 1.0.70 to 1.0.71 (#429)

* Bump actions/cache from 2.1.6 to 2.1.7 (#431)

Bumps [actions/cache](https://github.com/actions/cache) from 2.1.6 to 2.1.7.
- [Release notes](https://github.com/actions/cache/releases)
- [Commits](actions/cache@v2.1.6...v2.1.7)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump futures from 0.3.17 to 0.3.18 (#432)

Bumps [futures](https://github.com/rust-lang/futures-rs) from 0.3.17 to 0.3.18.
- [Release notes](https://github.com/rust-lang/futures-rs/releases)
- [Changelog](https://github.com/rust-lang/futures-rs/blob/master/CHANGELOG.md)
- [Commits](rust-lang/futures-rs@0.3.17...0.3.18)

---
updated-dependencies:
- dependency-name: futures
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* runtime main & staking integration to node (#427)

* runtime main & staking integration to node

Signed-off-by: R.RajeshKumar <[email protected]>

* node rpc refactor

Signed-off-by: R.RajeshKumar <[email protected]>

* change in runtime detect logic

Signed-off-by: R.RajeshKumar <[email protected]>

* Review comments closure

Signed-off-by: R.RajeshKumar <[email protected]>

* cargo fmt

Signed-off-by: R.RajeshKumar <[email protected]>

* nodle-chain version correction

Signed-off-by: R.RajeshKumar <[email protected]>

* bump version for release

* fix release script

* nodle parachain bringup (#433)

* also log into docker hub (#435)

* also log into docker hub

* rename workflow

* pallet grants BlockNumberProvider integration (#436)

Signed-off-by: R.RajeshKumar <[email protected]>

* minimize chain spec parameters (#437)

* create eden-local chain spec (#438)

* fix spec import (#440)

* Enable endowed accounts (#441)

* production eden spec (#442)

* generalize session key injection to aura and collator

* no longer inject session keys via curl

* kill chaos config

* move dockerfile up

* minimum production chain spec

* mount chain spec in node

* fine tune chain spec

* add placeholder bootnode

* add call filter

* bump version for release

* generate compressed runtime as well

* fix parachain id in genesis config

* remove log

* Bump serde from 1.0.130 to 1.0.131 (#443)

* pallets: reorganize benchmark compilation flags (#445)

* feat: support try-runtime for all our runtimes (#444)

* Bump serde from 1.0.131 to 1.0.132 (#446)

Bumps [serde](https://github.com/serde-rs/serde) from 1.0.131 to 1.0.132.
- [Release notes](https://github.com/serde-rs/serde/releases)
- [Commits](serde-rs/serde@v1.0.131...v1.0.132)

---
updated-dependencies:
- dependency-name: serde
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump futures from 0.3.18 to 0.3.19 (#447)

Bumps [futures](https://github.com/rust-lang/futures-rs) from 0.3.18 to 0.3.19.
- [Release notes](https://github.com/rust-lang/futures-rs/releases)
- [Changelog](https://github.com/rust-lang/futures-rs/blob/master/CHANGELOG.md)
- [Commits](rust-lang/futures-rs@0.3.18...0.3.19)

---
updated-dependencies:
- dependency-name: futures
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump serde from 1.0.132 to 1.0.133 (#448)

Bumps [serde](https://github.com/serde-rs/serde) from 1.0.132 to 1.0.133.
- [Release notes](https://github.com/serde-rs/serde/releases)
- [Commits](serde-rs/serde@v1.0.132...v1.0.133)

---
updated-dependencies:
- dependency-name: serde
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat(pallet-allocations): fast return on zero allocation (#450)

* feat: fast return on zero allocation

* test: fast return on zero allocation

* fix(ci): remove dead code (#451)

Installing ubuntu native dependencies had become dead as of a previous process simplification. It turns out CI runs fine without it.

* Upgrade rust edition (#449)

* fix: rust edition to 2021

* feat: update to polkadot v0.9.13 to fix ChainSpecExtention macro issue

* fix: remove light-client support

light-client is experiencing major change in the upstream. It's also not supported for para-chains yet

* fix: wiring of the new slash parameter disable_strategy

The parameter is not yet consumed properly in our slash computation. The proper support will come in NodleCode/chain-workspace#95

* fix: resolving para-id from cli. Use default para-id (1000) with "eden-local" and "eden-dev".

The id/para-id is not an explicit run command for a parachain anymore. It's assumed the parachain id is part of the extended chain spec. So for example no need to specify both "eden" and its id. "eden" should have a known id.

* fix(ci): specify the main-net wss port explicitly.

* minor refactorings (#454)

* Staking proto chainspec & verification updates ... (#453)

* staking proto chainspec & verification updates

Signed-off-by: shamb0 <[email protected]>

* cargo fmt fix

Signed-off-by: shamb0 <[email protected]>

* review fix

Signed-off-by: shamb0 <[email protected]>

* review fix

Signed-off-by: rajesh-nodle <[email protected]>

Co-authored-by: shamb0 <[email protected]>

* update copyright year from 2020 to 2022 (#460)

* preinstall toolchain in devcontainer (#473)

* replace arcadia chain spec (#474)

Co-authored-by: rajesh-nodle <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Alex Sedighi <[email protected]>
Co-authored-by: Nicholas Young <[email protected]>
Co-authored-by: shamb0 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant