-
Notifications
You must be signed in to change notification settings - Fork 26
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
yath-runner retry.t did not respond to SIGTERM, sending SIGKILL... #207
Comments
In an attempt at playing with some variables, I changed the |
I will leave the bug open for a while to see if anyone else has issues. If nothing else gets reported I may just close it. I am refactoring the code that would be responsible for killing stalled processes, so this will probably be fixed by that anyway. |
I (and most probably rjbs) encountered this issue while testing PAUSE Web with yath-runner under GitHub Action. Excerpt from a log ( https://github.com/charsbar/pause/actions/runs/4894552162/jobs/8738930578 )
SIGTERM comes from Test::mysqld ( https://metacpan.org/dist/Test-mysqld/source/lib/Test/mysqld.pm#L150-166 ) Hope this helps a bit. |
The issue here will be that yath is running as pid 1 in the container, which can ignore signals. We need to put a process between it and the root of the process tree. The usual way to do this is to docker run with |
When trying to install Test2::Harness in a docker container, I get an inconsistent failure. Sometimes it completes successfully (in about 8 minutes), and sometimes it fails after a half-hour of trying. I recently tried running the install as
cpanm --verbose Test2::Harness
, and it is appearing to get stuck in a loop, repeatedly saying for the past hour-plus:(INTERNAL) 2837 yath-runner /root/.cpanm/work/1607030892.1/Test2-Harness-1.000042/t/integration/retry.t did not respond to SIGTERM, sending SIGKILL to 3096...
I haven't tried with a minimal
Dockerfile
, but I'm using centos:latest as the base, installinggcc
,git
, andperl-App-cpanminus
viayum
, and then installing a bunch of modules viacpanm
, of which one isTest2::Harness
.For the record, the test before this,
failure_cases.t
, completes successfully.The text was updated successfully, but these errors were encountered: