Skip to content

Remove unused Rococo parachains runtimes#10648

Merged
acatangiu merged 3 commits intoparitytech:masterfrom
acatangiu:remove-rococo-unused-chains
Dec 15, 2025
Merged

Remove unused Rococo parachains runtimes#10648
acatangiu merged 3 commits intoparitytech:masterfrom
acatangiu:remove-rococo-unused-chains

Conversation

@acatangiu
Copy link
Copy Markdown
Contributor

Rococo has been decomissioned some while ago. We still kept the runtimes for Rococo<>Westend bridge zombienet and integration tests. This commit removes the unused runtimes and obsolete integration tests.

Removed:

  • coretime-rococo
  • people-rococo

Kept:

  • asset-hub-rococo
  • bridge-hub-rococo
  • rococo relay chain

Which are still being used/useful for R<>W bridge testing.

Also remove RAH integration tests which are obsolete. WAH integration tests cover all AH usecases.

This cleanup decreases maintenance burden (all FRAME changes had to be integrated to these defunct runtimes) and CI overhead: fewer tests to run.

@acatangiu acatangiu self-assigned this Dec 15, 2025
@acatangiu acatangiu added R0-no-crate-publish-required The change does not require any crates to be re-published. T10-tests This PR/Issue is related to tests. T14-system_parachains This PR/Issue is related to system parachains. labels Dec 15, 2025
Rococo has been decomissioned some while ago. We still kept the runtimes
for Rococo<>Westend bridge zombienet and integration tests. This commit
removes the unused runtimes and obsolete integration tests.

Removed:
- coretime-rococo
- people-rococo

Kept:
- asset-hub-rococo
- bridge-hub-rococo
- rococo relay chain

Which are still being used/useful for R<>W bridge testing.

Also remove RAH integration tests which are obsolete. WAH integration tests
cover all AH usecases.

This cleanup decreases maintenance burden (all FRAME changes had to be
integrated to these defunct runtimes) and CI overhead: fewer tests to run.

Signed-off-by: Adrian Catangiu <adrian@parity.io>
@acatangiu acatangiu force-pushed the remove-rococo-unused-chains branch from 2d60dfb to ee2d9e2 Compare December 15, 2025 12:33
@acatangiu acatangiu requested review from a team as code owners December 15, 2025 12:33
@github-actions
Copy link
Copy Markdown
Contributor

The PR removes unused Rococo runtimes and integration tests, which is good cleanup. However, there are several lingering references to the removed runtimes in cumulus/polkadot-parachain (the node binary) and polkadot/zombienet-sdk-tests. These references will cause compilation errors because the runtime crates have been removed from the workspace and dependencies. You need to remove the code that depends on these runtimes.


Review Suggestions

cumulus/polkadot-parachain/src/chain_spec/mod.rs:33, 59, 205-206, 335-336

This file still references rococo_parachain module and rococo_parachain_runtime (indirectly or in tests), which has been removed. The rococo_parachain module declaration and usage should be removed. The fallback to rococo_parachain_local_config also needs to be removed or updated.


cumulus/polkadot-parachain/src/chain_spec/people.rs:67-79, 118-149

The PeopleRuntimeType::Rococo* variants and the rococo module reference people_rococo_runtime, which has been removed. These should be removed.


cumulus/polkadot-parachain/src/chain_spec/coretime.rs:110-116, 144-184

The CoretimeRuntimeType::Rococo* variants and the rococo module reference coretime_rococo_runtime, which has been removed. These should be removed.


cumulus/polkadot-parachain/src/chain_spec/rococo_parachain.rs:1-78

This file uses rococo_parachain_runtime, which has been removed. This file should likely be deleted.


polkadot/zombienet-sdk-tests/build.rs:135

coretime-rococo is still in the CHAINS list. This will cause the build script to try and build coretime-rococo-runtime, which will fail.


polkadot/zombienet-sdk-tests/tests/smoke/coretime_revenue.rs:14-15, 245

This test relies on coretime-rococo metadata and chain spec. Since coretime-rococo is being removed, this test should probably be removed or adapted.


polkadot/zombienet-sdk-tests/tests/elastic_scaling/doesnt_break_parachains.rs:50

The comment references rococo-parachain. Verify if the test logic relies on rococo-parachain behavior or if it needs adjustment.

@acatangiu acatangiu added this pull request to the merge queue Dec 15, 2025
Merged via the queue into paritytech:master with commit 6504dbd Dec 15, 2025
239 of 252 checks passed
@acatangiu acatangiu deleted the remove-rococo-unused-chains branch December 15, 2025 14:49
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. T10-tests This PR/Issue is related to tests. T14-system_parachains This PR/Issue is related to system parachains.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants