Skip to content

Only mess with coretime if we are registering an actual parachain.#6554

Merged
ordian merged 4 commits intomasterfrom
rk-fix-para-registration
Nov 22, 2024
Merged

Only mess with coretime if we are registering an actual parachain.#6554
ordian merged 4 commits intomasterfrom
rk-fix-para-registration

Conversation

@eskimor
Copy link
Copy Markdown
Member

@eskimor eskimor commented Nov 20, 2024

No description provided.

@paritytech-workflow-stopper
Copy link
Copy Markdown

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/11932296679
Failed job name: fmt

@eskimor eskimor added R0-no-crate-publish-required The change does not require any crates to be re-published. T2-pallets This PR/Issue is related to a particular pallet. labels Nov 20, 2024
Comment thread polkadot/runtime/common/src/paras_sudo_wrapper.rs
@eskimor eskimor enabled auto-merge November 20, 2024 12:08
@eskimor eskimor added this pull request to the merge queue Nov 20, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 20, 2024
@ordian ordian enabled auto-merge November 22, 2024 12:30
@ordian ordian added this pull request to the merge queue Nov 22, 2024
Merged via the queue into master with commit 08ec8cd Nov 22, 2024
@ordian ordian deleted the rk-fix-para-registration branch November 22, 2024 14:11
Krayt78 pushed a commit to Krayt78/polkadot-sdk that referenced this pull request Dec 18, 2024
…aritytech#6554)

Co-authored-by: Robert <robert@gonimo.com>
Co-authored-by: ordian <write@reusable.software>
dudo50 pushed a commit to paraspell-research/polkadot-sdk that referenced this pull request Jan 4, 2025
…aritytech#6554)

Co-authored-by: Robert <robert@gonimo.com>
Co-authored-by: ordian <write@reusable.software>
github-merge-queue Bot pushed a commit that referenced this pull request Jun 20, 2025
Rewrite it with zombienet-sdk, while simplyfing it.
The main source of flakyness was timing, because this test was manually
registering parachains, so we had to wait for 2 sessions. Waiting for
session change with zndsl zombienet was a hassle, whereas with
zombienet-sdk is much easier.

Moreover, I needed to replicate the logic change from:
#6554 to the genesis
parachain registration, so that we don't automatically get an extra
assigned core when registering a parachain (unless we want one)
alvicsam pushed a commit that referenced this pull request Oct 17, 2025
Rewrite it with zombienet-sdk, while simplyfing it.
The main source of flakyness was timing, because this test was manually
registering parachains, so we had to wait for 2 sessions. Waiting for
session change with zndsl zombienet was a hassle, whereas with
zombienet-sdk is much easier.

Moreover, I needed to replicate the logic change from:
#6554 to the genesis
parachain registration, so that we don't automatically get an extra
assigned core when registering a parachain (unless we want one)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

R0-no-crate-publish-required The change does not require any crates to be re-published. T2-pallets This PR/Issue is related to a particular pallet.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants