-
Notifications
You must be signed in to change notification settings - Fork 784
[CI] Revert w/a for FPGA emu bug #5775
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
Conversation
Timed Out : 8 Do you know what might cause this issue? |
One failed test is a known issue, 8 timed out tests require investigation |
Is it related to this change? If no, please, disable the failing test to avoid false alarms.
The same - please, check that these are not related to your patch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, fix pre-commit fails.
Looking into it |
FYI. I see something related in pre-commit for #5778. See https://github.com/intel/llvm/runs/5496313460?check_suite_focus=true. Timed Out Tests (7): Testing Time: 1654.86s |
Another one - https://github.com/intel/llvm/runs/5496822497?check_suite_focus=true. |
Another instance: https://github.com/intel/llvm/runs/5503076361?check_suite_focus=true. Pre-commit for #5772. |
A different picture here: https://github.com/intel/llvm/runs/5504737044?check_suite_focus=true
|
@bader I am trying to reproduce timeout failures using intel/llvm by running whole llvm-test-suite multiple times with -j 12, but no luck so far. Actually I was doing this during the last week to test this patch and didn't face timeout issues. I don't have information about how many testing jobs we run on the same machine, how many threads do we use (-j threads]) and if smth changed recently in the why we run testing jobs in CI. Also I need an access to one of the CI machines where problem is reproduced. Who can provide me this information? |
@bader I've created a PR #5794 which reverts #5231. 5 tests failed wih timeout: I hope this proves that #5231 is not a guilty commit for timeout failures. |
Containers were updated with fixed FPGA emulator packages.