Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-22249

Record process ID of spawned process

XMLWordPrintable

      Currently BourneShellScript does not try to track the PID of the external process; it assumes that it runs to completion (successfully or not) and then writes the status file. But if the wrapping shell script is killed, or the machine rebooted, this might never occur and the task would hang.

      Better to record $$ from the outer sh process and override exitStatus in case no such process still exists. Implies some use of native code.

      (TBD whether an analogous fix can be done for Windows as well.)

          [JENKINS-22249] Record process ID of spawned process

          Jesse Glick created issue -
          Jesse Glick made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Jesse Glick made changes -
          Link New: This issue is related to JENKINS-25053 [ JENKINS-25053 ]
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          Jesse Glick made changes -
          Resolution Original: Fixed [ 1 ]
          Status Original: Resolved [ 5 ] New: Reopened [ 4 ]
          Jesse Glick made changes -
          Status Original: Reopened [ 4 ] New: Open [ 1 ]
          Jesse Glick made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          Jesse Glick made changes -
          Link New: This issue depends on JENKINS-25727 [ JENKINS-25727 ]
          Jesse Glick made changes -
          Link New: This issue depends on JENKINS-32264 [ JENKINS-32264 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 154329 ] New: JNJira + In-Review [ 194870 ]

            jglick Jesse Glick
            jglick Jesse Glick
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: