Channel listeners onClose method would propagate exceptions causing extra noise when failing to close the channel.

          [JENKINS-28062] Channel listener onClose propagated exceptions

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          src/main/java/hudson/remoting/Channel.java
          http://jenkins-ci.org/commit/remoting/b60df6b39caa403f29845b86338579d1cf76b819
          Log:
          [FIXED JENKINS-28062] A Channel.Listener's onClose method that propagates an exception is a sign of a bad listener not a problem closing the channel.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: src/main/java/hudson/remoting/Channel.java http://jenkins-ci.org/commit/remoting/b60df6b39caa403f29845b86338579d1cf76b819 Log: [FIXED JENKINS-28062] A Channel.Listener's onClose method that propagates an exception is a sign of a bad listener not a problem closing the channel.

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          src/main/java/hudson/remoting/Channel.java
          http://jenkins-ci.org/commit/remoting/846b3e0527aa1dc5629a2a7e870f4b7efb653cfb
          Log:
          Merge pull request #42 from jenkinsci/safe-listener-fire

          [FIXED JENKINS-28062] A Channel.Listener's onClose method that propagate...

          Compare: https://github.com/jenkinsci/remoting/compare/2b6a02192ca4...846b3e0527aa

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: src/main/java/hudson/remoting/Channel.java http://jenkins-ci.org/commit/remoting/846b3e0527aa1dc5629a2a7e870f4b7efb653cfb Log: Merge pull request #42 from jenkinsci/safe-listener-fire [FIXED JENKINS-28062] A Channel.Listener's onClose method that propagate... Compare: https://github.com/jenkinsci/remoting/compare/2b6a02192ca4...846b3e0527aa

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          core/src/main/java/hudson/slaves/SlaveComputer.java
          http://jenkins-ci.org/commit/jenkins/d760216d1e3a927b971edf27c7af317a5de71e20
          Log:
          [FIXED JENKINS-28062] A Launcher.afterDisconnect() method that propagates an exception is a sign of a bad Launcher not a problem closing the channel.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/java/hudson/slaves/SlaveComputer.java http://jenkins-ci.org/commit/jenkins/d760216d1e3a927b971edf27c7af317a5de71e20 Log: [FIXED JENKINS-28062] A Launcher.afterDisconnect() method that propagates an exception is a sign of a bad Launcher not a problem closing the channel.

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          core/src/main/java/hudson/slaves/SlaveComputer.java
          http://jenkins-ci.org/commit/jenkins/5e311a3dca6c725ab794dc35504bf0484a56be8f
          Log:
          Merge pull request #1669 from stephenc/jenkins-28062

          JENKINS-28062 Channel listener onClose propagated exceptions

          Compare: https://github.com/jenkinsci/jenkins/compare/3bf606178640...5e311a3dca6c

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/java/hudson/slaves/SlaveComputer.java http://jenkins-ci.org/commit/jenkins/5e311a3dca6c725ab794dc35504bf0484a56be8f Log: Merge pull request #1669 from stephenc/jenkins-28062 JENKINS-28062 Channel listener onClose propagated exceptions Compare: https://github.com/jenkinsci/jenkins/compare/3bf606178640...5e311a3dca6c

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          changelog.html
          http://jenkins-ci.org/commit/jenkins/6a138b59b4eee42f65d620730802d3a97a0f6c4a
          Log:
          Noting JENKINS-28062

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: changelog.html http://jenkins-ci.org/commit/jenkins/6a138b59b4eee42f65d620730802d3a97a0f6c4a Log: Noting JENKINS-28062

          dogfood added a comment -

          Integrated in jenkins_main_trunk #4106
          [FIXED JENKINS-28062] A Launcher.afterDisconnect() method that propagates an exception is a sign of a bad Launcher not a problem closing the channel. (Revision d760216d1e3a927b971edf27c7af317a5de71e20)
          Noting JENKINS-28062 (Revision 6a138b59b4eee42f65d620730802d3a97a0f6c4a)

          Result = SUCCESS
          stephen connolly : d760216d1e3a927b971edf27c7af317a5de71e20
          Files :

          • core/src/main/java/hudson/slaves/SlaveComputer.java

          stephen connolly : 6a138b59b4eee42f65d620730802d3a97a0f6c4a
          Files :

          • changelog.html

          dogfood added a comment - Integrated in jenkins_main_trunk #4106 [FIXED JENKINS-28062] A Launcher.afterDisconnect() method that propagates an exception is a sign of a bad Launcher not a problem closing the channel. (Revision d760216d1e3a927b971edf27c7af317a5de71e20) Noting JENKINS-28062 (Revision 6a138b59b4eee42f65d620730802d3a97a0f6c4a) Result = SUCCESS stephen connolly : d760216d1e3a927b971edf27c7af317a5de71e20 Files : core/src/main/java/hudson/slaves/SlaveComputer.java stephen connolly : 6a138b59b4eee42f65d620730802d3a97a0f6c4a Files : changelog.html

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          core/src/main/java/hudson/slaves/SlaveComputer.java
          http://jenkins-ci.org/commit/jenkins/d42e8465fc2829bc7a926fe8d52d3bdfaabd857e
          Log:
          [FIXED JENKINS-28062] A Launcher.afterDisconnect() method that propagates an exception is a sign of a bad Launcher not a problem closing the channel.

          (cherry picked from commit d760216d1e3a927b971edf27c7af317a5de71e20)

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/java/hudson/slaves/SlaveComputer.java http://jenkins-ci.org/commit/jenkins/d42e8465fc2829bc7a926fe8d52d3bdfaabd857e Log: [FIXED JENKINS-28062] A Launcher.afterDisconnect() method that propagates an exception is a sign of a bad Launcher not a problem closing the channel. (cherry picked from commit d760216d1e3a927b971edf27c7af317a5de71e20)

          I do not think this can be 1.609.1-fixed, as it also requires remoting pull request 42 which hasn't been released yet.

          Steven Christou added a comment - I do not think this can be 1.609.1-fixed, as it also requires remoting pull request 42 which hasn't been released yet.

          dogfood added a comment -

          Integrated in jenkins_main_trunk #4292
          [FIXED JENKINS-28062] A Launcher.afterDisconnect() method that propagates an exception is a sign of a bad Launcher not a problem closing the channel. (Revision d42e8465fc2829bc7a926fe8d52d3bdfaabd857e)

          Result = UNSTABLE
          ogondza : d42e8465fc2829bc7a926fe8d52d3bdfaabd857e
          Files :

          • core/src/main/java/hudson/slaves/SlaveComputer.java

          dogfood added a comment - Integrated in jenkins_main_trunk #4292 [FIXED JENKINS-28062] A Launcher.afterDisconnect() method that propagates an exception is a sign of a bad Launcher not a problem closing the channel. (Revision d42e8465fc2829bc7a926fe8d52d3bdfaabd857e) Result = UNSTABLE ogondza : d42e8465fc2829bc7a926fe8d52d3bdfaabd857e Files : core/src/main/java/hudson/slaves/SlaveComputer.java

            Unassigned Unassigned
            schristou Steven Christou
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: