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

Exceptions during Jenkins#cleanup() should not block the restart logic

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • core

      It is a follow-up to https://github.com/jenkinsci/jenkins/pull/2746

      In Jenkins we call cleanup() handlers after destroying the WebAppMain context. At this point Jenkins is already semi-dead, hence we should proceed with the restart in any case even if the cleanup fails. Otherwise we may get a dead instance.

      I am still not sure about LifeCycles, but Jenkins code reviewers approved it. Will monitor the related issues

          [JENKINS-42164] Exceptions during Jenkins#cleanup() should not block the restart logic

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          core/src/main/java/hudson/WebAppMain.java
          core/src/main/java/hudson/lifecycle/SolarisSMFLifecycle.java
          core/src/main/java/hudson/lifecycle/UnixLifecycle.java
          core/src/main/java/hudson/lifecycle/WindowsServiceLifecycle.java
          http://jenkins-ci.org/commit/jenkins/4ab693846ca7a4aa112959a2e92688c3fb9122c3
          Log:
          Merge pull request #2746 from oleg-nenashev/bug/JENKINS-32820

          [JENKINS-32820, JENKINS-42164] - Windows service restart does not retain the build queue

          Compare: https://github.com/jenkinsci/jenkins/compare/1763e6400f56...4ab693846ca7

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: core/src/main/java/hudson/WebAppMain.java core/src/main/java/hudson/lifecycle/SolarisSMFLifecycle.java core/src/main/java/hudson/lifecycle/UnixLifecycle.java core/src/main/java/hudson/lifecycle/WindowsServiceLifecycle.java http://jenkins-ci.org/commit/jenkins/4ab693846ca7a4aa112959a2e92688c3fb9122c3 Log: Merge pull request #2746 from oleg-nenashev/bug/ JENKINS-32820 [JENKINS-32820, JENKINS-42164] - Windows service restart does not retain the build queue Compare: https://github.com/jenkinsci/jenkins/compare/1763e6400f56...4ab693846ca7

          Daniel Beck added a comment -

          Is this resolved by the merge?

          Daniel Beck added a comment - Is this resolved by the merge?

          Oleg Nenashev added a comment -

          Yes, released in 2.47

          Oleg Nenashev added a comment - Yes, released in 2.47

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          core/src/main/java/hudson/WebAppMain.java
          core/src/main/java/hudson/lifecycle/SolarisSMFLifecycle.java
          core/src/main/java/hudson/lifecycle/UnixLifecycle.java
          core/src/main/java/hudson/lifecycle/WindowsServiceLifecycle.java
          http://jenkins-ci.org/commit/jenkins/5f655a59b6d263061d6c5c1d05ac8d5d1d1d391f
          Log:
          Merge pull request #2746 from oleg-nenashev/bug/JENKINS-32820

          [JENKINS-32820, JENKINS-42164] - Windows service restart does not retain the build queue
          (cherry picked from commit 4ab693846ca7a4aa112959a2e92688c3fb9122c3)

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: core/src/main/java/hudson/WebAppMain.java core/src/main/java/hudson/lifecycle/SolarisSMFLifecycle.java core/src/main/java/hudson/lifecycle/UnixLifecycle.java core/src/main/java/hudson/lifecycle/WindowsServiceLifecycle.java http://jenkins-ci.org/commit/jenkins/5f655a59b6d263061d6c5c1d05ac8d5d1d1d391f Log: Merge pull request #2746 from oleg-nenashev/bug/ JENKINS-32820 [JENKINS-32820, JENKINS-42164] - Windows service restart does not retain the build queue (cherry picked from commit 4ab693846ca7a4aa112959a2e92688c3fb9122c3)

            Unassigned Unassigned
            oleg_nenashev Oleg Nenashev
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: