-
Bug
-
Resolution: Duplicate
-
Major
If you run a flow
node {
sh 'rm -rf * .*'
}
this deletes the control directory for the shell step while it is running, and it becomes impossible to abort the build.
I think this is because FileMonitoringController.exitStatus just checks if jenkins-result.txt exists yet. But if the whole control dir is missing, it should return -1 or similar.
Similar effect if you botch the $PATH so that echo cannot be found. In that case the control dir is present, but pid is not, so the controller assumes the process is still starting! It should impose a time limit on the startup phase.
Was ameliorated by improving DurableTaskExecution.stop to do a hard kill if you try to stop the build twice. This makes the similar JENKINS-25678 much less of a problem.
- is duplicated by
-
JENKINS-27105 workflow-plugin: 'env' before 'sh' will hang 'sh'
-
- Resolved
-
- is related to
-
JENKINS-25678 BatchController subject to space-in-path bug
-
- Resolved
-
-
JENKINS-28400 Not timing out when launch pid fails to appear
-
- Resolved
-
-
JENKINS-25550 Hard kill
-
- Resolved
-
-
JENKINS-27152 Store sh control files outside of workspace
-
- Resolved
-
Current BourneShellScript specifically declines to copy stdout/stderr from the wrapper process to the build log, since doing so would consume a Thread, but this also makes diagnosis of this class of error harder. Should at a minimum allow the diagnosis to be enabled with a system property. Maybe it is possible to allocate a thread which stops listening after one second. Or maybe up to (say) five concurrent tasks in the system would get this diagnosis applied but subsequent ones would not, so that people experimenting with flow definitions would get the benefit of diagnosis but there would be little performance hit in busy production systems.