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.

Sign up for GitHub

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

Jump to bottom

Fix flaky zombienet test zombienet-polkadot-smoke-0004-coretime-smoke-test #6226

Closed
Tracked by #5852
pepoviola opened this issue Oct 24, 2024 · 4 comments · Fixed by #6236
Closed
Tracked by #5852

Fix flaky zombienet test zombienet-polkadot-smoke-0004-coretime-smoke-test #6226

pepoviola opened this issue Oct 24, 2024 · 4 comments · Fixed by #6236
Labels
T10-tests This PR/Issue is related to tests.

Comments

@pepoviola
Copy link
Contributor

Some examples from yesterday ( 32 of 57 fails)

https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7628951
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7628817
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7628682
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7628615
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7628483
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7628416
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7628285
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7628229
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7627210
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7627074
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7626873
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7626039
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7625972
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7625769
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7625639
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7625576
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7625564
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7624714
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7624291
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7624224
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7623747
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7623613
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7623345
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7623211

cc @paritytech/parachains-core
@pepoviola pepoviola added the T10-tests This PR/Issue is related to tests. label Oct 24, 2024
@pepoviola pepoviola mentioned this issue Oct 24, 2024
Open
16 tasks
@ordian
Copy link
Member

ordian commented Oct 24, 2024

cc @eskimor @tdimitrov

pepoviola added a commit that referenced this issue Oct 24, 2024
@pepoviola
disable flaky test reported in #6226
77fae50
@tdimitrov
Copy link
Contributor

Artefacts are starting to get removed so I persisted the ones from:

In both cases the problem seems to be:

TRACE tokio-runtime-worker aura::cumulus: [Parachain] Para is not scheduled on any core, skipping import notification relay_parent=0x57c0a50d2a7e14237c9f1215b6797ff009fe4faa929698fa1ea0f766da7dd3ad params.para_id=Id(100)

I'm checking further why no cores are assigned to the para.

@pepoviola
Copy link
Contributor Author

Thanks @tdimitrov!

@pepoviola pepoviola mentioned this issue Oct 25, 2024
Closed
@tdimitrov tdimitrov mentioned this issue Oct 25, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Oct 28, 2024
@tdimitrov
Fix a flaky zombienet test - 0004-coretime-smoke-test (#6236)
df66d76 
In the test log I noticed that the batch transaction which configures
the coretime chain fails. However when rerunning the transaction
manually - it worked. Then I noticed that the coretime chain is
initially registered via zombienet and then re-registered via
`0004-configure-relay.js`. Because of this there is a period of time
when it's not producing blocks and `0004-configure-broker.js` fails to
setup the coretime chain. My theory is that the transaction has failed
because the coretime chain is stalled during the re-registration.

Fixes #6226
@pepoviola
Copy link
Contributor Author

Awesome! Thanks @tdimitrov 🙌

xermicus pushed a commit that referenced this issue Oct 28, 2024
@tdimitrov @xermicus
Fix a flaky zombienet test - 0004-coretime-smoke-test (#6236)
b06f99a 
In the test log I noticed that the batch transaction which configures
the coretime chain fails. However when rerunning the transaction
manually - it worked. Then I noticed that the coretime chain is
initially registered via zombienet and then re-registered via
`0004-configure-relay.js`. Because of this there is a period of time
when it's not producing blocks and `0004-configure-broker.js` fails to
setup the coretime chain. My theory is that the transaction has failed
because the coretime chain is stalled during the re-registration.

Fixes #6226
@tdimitrov tdimitrov mentioned this issue Oct 29, 2024
Merged
mordamax pushed a commit to paritytech-stg/polkadot-sdk that referenced this issue Oct 29, 2024
@tdimitrov @mordamax
Fix a flaky zombienet test - 0004-coretime-smoke-test (paritytech#6236)
0c6964c 
In the test log I noticed that the batch transaction which configures
the coretime chain fails. However when rerunning the transaction
manually - it worked. Then I noticed that the coretime chain is
initially registered via zombienet and then re-registered via
`0004-configure-relay.js`. Because of this there is a period of time
when it's not producing blocks and `0004-configure-broker.js` fails to
setup the coretime chain. My theory is that the transaction has failed
because the coretime chain is stalled during the re-registration.

Fixes paritytech#6226
mordamax pushed a commit to paritytech-stg/polkadot-sdk that referenced this issue Oct 29, 2024
@tdimitrov @mordamax
Fix a flaky zombienet test - 0004-coretime-smoke-test (paritytech#6236)
0e4bcce 
In the test log I noticed that the batch transaction which configures
the coretime chain fails. However when rerunning the transaction
manually - it worked. Then I noticed that the coretime chain is
initially registered via zombienet and then re-registered via
`0004-configure-relay.js`. Because of this there is a period of time
when it's not producing blocks and `0004-configure-broker.js` fails to
setup the coretime chain. My theory is that the transaction has failed
because the coretime chain is stalled during the re-registration.

Fixes paritytech#6226
github-merge-queue bot pushed a commit that referenced this issue Oct 30, 2024
@tdimitrov @actions-user @ggwpez
Bump a timeout in zombienet coretime smoke test (#6268)
4e24733 
polkadot/zombienet_tests/smoke/0004-coretime-smoke-test.zndsl still
timeouts on CI from time to time. Bumping the timeout a bit more.

Related to #6226

---------

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Co-authored-by: GitHub Action <[email protected]>
Co-authored-by: Oliver Tale-Yazdi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T10-tests This PR/Issue is related to tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix a flaky zombienet test - 0004-coretime-smoke-test paritytech/polkadot-sdk
3 participants
@pepoviola @tdimitrov @ordian