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

Evaluate integration test stability #233

Closed
clangenb opened this issue Jan 22, 2025 · 1 comment
Closed

Evaluate integration test stability #233

clangenb opened this issue Jan 22, 2025 · 1 comment

Comments

@clangenb
Copy link
Member

clangenb commented Jan 22, 2025

We have a trade-off between CI execeution time vs. Stability and the tuning parameter is the waiting-blocks after an extrinsic to ensure that our extrinsic is in the canonical chain.

Initially, it looked like 5 blocks of waiting is enough, but now we do get some failures due to tx reordering (i.e., next-phase accidentally before registering). Maybe it makes sense to increase the number to 7.

EDIT: Current CI fail rate seems to be around 25%

@clangenb
Copy link
Member Author

As mentioned here, #234 (comment), I believe that we have a reasonable stability trade-off and we just hope that future substrate updates can make our CI more stable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant