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

Jenkins does not invoke ProcessKillers for Windows recursively

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Major Major
    • core

      Currently, On Windows Jenkins calls ProcessKillers for the root process only. On Linux machines the extension point is being called for the entire process tree. The implementation should be synchronized for all platforms.

      Solution:

      • Built-in killRecursive() of WinProc should be replaced by a recursive implementation from UnixProcess (depends on JENKINS-20289)

          [JENKINS-19156] Jenkins does not invoke ProcessKillers for Windows recursively

          Oleg Nenashev created issue -

          Oleg Nenashev added a comment -

          It is not possible to implement Cygwin Process Killer

          Oleg Nenashev added a comment - It is not possible to implement Cygwin Process Killer
          Oleg Nenashev made changes -
          Link New: This issue is blocking JENKINS-13615 [ JENKINS-13615 ]

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          core/src/main/java/hudson/util/ProcessTree.java
          http://jenkins-ci.org/commit/jenkins/20859b4aad63689b5338fa32f50cb731118bf16f
          Log:
          Windows::killRecursively() invokes process killers after completion.
          Resolves https://issues.jenkins-ci.org/browse/JENKINS-19156

          Signed-off-by: Oleg Nenashev <nenashev@synopsys.com>

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: core/src/main/java/hudson/util/ProcessTree.java http://jenkins-ci.org/commit/jenkins/20859b4aad63689b5338fa32f50cb731118bf16f Log: Windows::killRecursively() invokes process killers after completion. Resolves https://issues.jenkins-ci.org/browse/JENKINS-19156 Signed-off-by: Oleg Nenashev <nenashev@synopsys.com>
          Oleg Nenashev made changes -
          Labels New: lts-candidate

          Oleg Nenashev added a comment -

          Previous fix invokes ProcessKiller only for top-level process.
          Built-in killRecursive() of WinProc should be replaced by manual implementation from UnixProcess

          Oleg Nenashev added a comment - Previous fix invokes ProcessKiller only for top-level process. Built-in killRecursive() of WinProc should be replaced by manual implementation from UnixProcess
          Oleg Nenashev made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Oleg Nenashev made changes -
          Link New: This issue depends on JENKINS-20289 [ JENKINS-20289 ]
          Daniel Beck made changes -
          Link New: This issue is related to JENKINS-9104 [ JENKINS-9104 ]

          Daniel Beck added a comment -

          JENKINS-9104 seems to indicate that processes are getting killed. What is this issue about, specifically?

          Daniel Beck added a comment - JENKINS-9104 seems to indicate that processes are getting killed. What is this issue about, specifically?

            Unassigned Unassigned
            oleg_nenashev Oleg Nenashev
            Votes:
            3 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated: