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

[CI]: fix commands being stuck problem #3996

Open
apostasie opened this issue Mar 7, 2025 · 1 comment · May be fixed by #4056
Open

[CI]: fix commands being stuck problem #3996

apostasie opened this issue Mar 7, 2025 · 1 comment · May be fixed by #4056
Assignees
Labels
area/ci e.g., CI failure
Milestone

Comments

@apostasie
Copy link
Contributor

apostasie commented Mar 7, 2025

Some commands being tested do deadlock sometimes.
This is particularly true for EL8, for yet undiagnosed reasons.

Because we do not timeout on command execution, the runner will just kill everything, and we do not get any information about which command (nor which test) did actually deadlock, and no logs of course.

We should be able to have some command execution default deadline in place at the test tooling level.
That way, commands being stuck for more than say, 5 minutes, should break and we would report a failed "timed-out test" along with appropriate logs.

@AkihiroSuda can you assign this to me please?

@apostasie apostasie added the kind/unconfirmed-bug-claim Unconfirmed bug claim label Mar 7, 2025
@AkihiroSuda AkihiroSuda added area/ci e.g., CI failure and removed kind/unconfirmed-bug-claim Unconfirmed bug claim labels Mar 7, 2025
@apostasie
Copy link
Contributor Author

@fahedouch can you slate this for milestone 2.x.x?

@AkihiroSuda AkihiroSuda added this to the v2.x.x milestone Mar 20, 2025
@apostasie apostasie changed the title CI: fix commands being stuck problem [CI]: fix commands being stuck problem Apr 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci e.g., CI failure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants