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

Test sync::syncs_header_only_forks is flaky in sc-network-test #497

Open
dmitry-markin opened this issue Jul 24, 2023 · 2 comments
Open
Labels
T10-tests This PR/Issue is related to tests.

Comments

@dmitry-markin
Copy link
Contributor

dmitry-markin commented Jul 24, 2023

With changes from paritytech/substrate#14337, test syncs_header_only_forks is flaky on a multi-threaded runtime. This is only an issue on some machines where it fails ~ every 4 hours of continuous running with other sync tests.

For some reason, the test is not flaky if run under single-threaded #[tokio::test] runtime. So, paritytech/substrate#14337 converts it into single-threaded. Nevertheless, this needs some attention, so marked as TODO in the code.

See paritytech/substrate#14337 (comment) for additional details.

@ggwpez
Copy link
Member

ggwpez commented Jul 25, 2023

Is it using thread-local state somewhere?

@dmitry-markin
Copy link
Contributor Author

Is it using thread-local state somewhere?

That's highly unlikely.

@altonen altonen transferred this issue from paritytech/substrate Aug 24, 2023
@the-right-joyce the-right-joyce added T10-tests This PR/Issue is related to tests. and removed I5-tests labels Aug 25, 2023
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 10, 2024
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T10-tests This PR/Issue is related to tests.
Projects
Status: Backlog 🗒
Development

No branches or pull requests

3 participants