Skip to content

rewrite systematic chunk recovery test with zombienet-sdk#8553

Closed
alindima wants to merge 1 commit into
masterfrom
alindima/systematic-chunk-test
Closed

rewrite systematic chunk recovery test with zombienet-sdk#8553
alindima wants to merge 1 commit into
masterfrom
alindima/systematic-chunk-test

Conversation

@alindima

Copy link
Copy Markdown
Contributor

Rewrite systematic chunk recovery test with zombienet-sdk.

Also switches glutton-westend back to 6-second block time, which is why this test must have been failing sometimes (since it had a 2-second slot but wasn't using the slot-based collator). This was changed by me in #5372 (probably for some versi testing and I later forgot to remove it)

@alindima alindima requested review from a team as code owners May 16, 2025 13:58
@alindima alindima added T8-polkadot This PR/Issue is related to/affects the Polkadot network. T10-tests This PR/Issue is related to tests. labels May 16, 2025
@paritytech-review-bot paritytech-review-bot Bot requested a review from a team May 16, 2025 14:28
@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/15070197966
Failed job name: cargo-clippy

}))
.with_collator(|n| n.with_name("collator-2000"))
})
.with_parachain(|p| {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems copy paste from above, dedup ?

type MaxAuthorities = ConstU32<100_000>;
type AllowMultipleBlocksPerSlot = ConstBool<true>;
type SlotDuration = ConstU64<2000>;
type SlotDuration = ConstU64<SLOT_DURATION>;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand why we need to make this change. Why not run it with the slot based collator ?

Should do 6s blocks just fine as is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T8-polkadot This PR/Issue is related to/affects the Polkadot network. T10-tests This PR/Issue is related to tests.

Projects

Status: Completed

Development

Successfully merging this pull request may close these issues.

4 participants