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

Backport: Fix multiple cores assignments 1.7 (#3469) #3497

Merged
merged 4 commits into from
Feb 28, 2024

Conversation

bkontur
Copy link
Contributor

@bkontur bkontur commented Feb 27, 2024

Backport from release-polkadot-v1.7.2.

Expected patches for (1.7.0):

  • polkadot-node-core-backing 8.0.1

  • polkadot-node-core-provisioner 8.0.1

  • polkadot-statement-table 8.0.1

  • polkadot-primitives 8.0.1

  • polkadot-runtime-parachains 8.0.1

  • polkadot-service 8.0.1

First step into elastic scaling: Having a para be on multiple cores,
will not get it more throughput yet, but it will no longer brick it.

This is a backport PR, as this would benefit the Coretime launch on
Kusama.

---------

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>
Signed-off-by: alindima <alin@parity.io>
Co-authored-by: Andrei Sandu <54316454+sandreim@users.noreply.github.com>
Co-authored-by: alindima <alin@parity.io>
Co-authored-by: eskimor <eskimor@no-such-url.com>
@bkontur bkontur added the R0-silent Changes should not be mentioned in any release notes label Feb 27, 2024
@bkontur bkontur requested a review from a team as a code owner February 27, 2024 21:57
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable-int
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/5372881

@paritytech-cicd-pr
Copy link

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

@bkchr
Copy link
Member

bkchr commented Feb 28, 2024

Duplicate of #3469?

@acatangiu acatangiu merged commit 0fa7f6f into release-crates-io-v1.7.0 Feb 28, 2024
123 of 127 checks passed
@acatangiu acatangiu deleted the bko-backport-coretime-1.7 branch February 28, 2024 10:29
bkontur added a commit to bkontur/runtimes that referenced this pull request Feb 28, 2024
bkontur added a commit to bkontur/runtimes that referenced this pull request Feb 28, 2024
fellowship-merge-bot bot pushed a commit to polkadot-fellows/runtimes that referenced this pull request Mar 5, 2024
Based on bump to
[`polkadot-sdk@1.6.0`](#159).

Attached result of `cargo upgrade -v --pinned --incompatible`
[cargo-upgrade-version-bump.log](https://github.com/polkadot-fellows/runtimes/files/14281072/cargo-upgrade-version-bump.log)

~~## For reviewers~~

~~This PR is against `polkadot-fellows`'s main to bring it to the
fellows repo, but if you want to see a real diff relevant to the
`polkadot-sdk@1.7.0` update please check:
bkontur/runtimes@bko-bump-to-1.6...bkontur:runtimes:bko-bump-to-1.7.~~


## TODO

- [x] fix compilation
- [x] apply patch for AssetHub tests:
paritytech/polkadot-sdk#3404
- [x] apply patch for `coretime`
paritytech/polkadot-sdk#3497 see
[comment](#187 (comment))
- [x] fix integration tests - fix after
#185
- [x] fix benchmarks (also try them)
- [x] apply patch for `pallet-xcm` benchmarks
paritytech/polkadot-sdk#3465
- [x] check ranked collectives:
- [x] `ExchangeOrigin` see
[comment](#187 (comment))
- [x] `MemberSwappedHandler` / `BenchmarkSetup` see
[comment](#187 (comment))

---------

Signed-off-by: Adrian Catangiu <adrian@parity.io>
Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com>
Co-authored-by: Francisco Aguirre <franciscoaguirreperez@gmail.com>
Co-authored-by: Ross Bulat <ross@parity.io>
Co-authored-by: Alain Brenzikofer <alain@integritee.network>
Co-authored-by: eskimor <eskimor@no-such-url.com>
Co-authored-by: Adrian Catangiu <adrian@parity.io>
Co-authored-by: brenzi <brenzi@users.noreply.github.com>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants