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

IF: Test Failure: nodeos_startup_catchup_if_lr_test #2351

Closed
heifner opened this issue Mar 28, 2024 · 2 comments
Closed

IF: Test Failure: nodeos_startup_catchup_if_lr_test #2351

heifner opened this issue Mar 28, 2024 · 2 comments
Assignees

Comments

@heifner
Copy link
Member

heifner commented Mar 28, 2024

AssertionError: Expected to at least receive 9.0 transactions per block, but only getting 7.6

It is not clear what is going on here. Could just mean our ci/cd could not keep up. Or maybe all the voting is slowing it down enough it can't process trxs quick enough. Would be worth looking closer at the logs. Might need more net threads to allow trx processing along with vote processing. We should consider processing votes in its own thread pool so they can't clog the net thread pool.

Happened at the same time:
https://github.com/AntelopeIO/leap/actions/runs/8467659168/job/23199344820 ubsan
https://github.com/AntelopeIO/leap/actions/runs/8467659168/job/23199345763 ubnuntu22repo

@greg7mdp
Copy link
Contributor

greg7mdp commented Apr 2, 2024

Issue was caused by the addition of two new tests in #2335.

Two workflow runs with these tests removed passed without problems. https://github.com/AntelopeIO/leap/actions/runs/8523025337

Resolution:

@spoonincode will beef up the instances the workflows are run on for: NP, LR, ubsan, asan, asserton
I have created issue #2369 to track this task.

@greg7mdp greg7mdp closed this as not planned Won't fix, can't repro, duplicate, stale Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

4 participants