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

Excessive classes being sent to slave machine

XMLWordPrintable

      The `Computer.getHostName` method sends the `Computer.class` which will send `Jenkins.class` for only logging. This can lead to slow Jenkins startups.

          [JENKINS-28058] Excessive classes being sent to slave machine

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          core/src/main/java/hudson/model/Computer.java
          http://jenkins-ci.org/commit/jenkins/df831325b6623c479dcf4b215415f4523d10530b
          Log:
          [FIXED JENKINS-28058] Don't send a reference to Computer.class over remoting channels

          • Logging added to help diagnose was actually causing Jenkins.class to be initialized on the remoting side

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/java/hudson/model/Computer.java http://jenkins-ci.org/commit/jenkins/df831325b6623c479dcf4b215415f4523d10530b Log: [FIXED JENKINS-28058] Don't send a reference to Computer.class over remoting channels Logging added to help diagnose was actually causing Jenkins.class to be initialized on the remoting side

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          core/src/main/java/hudson/model/Computer.java
          http://jenkins-ci.org/commit/jenkins/3bf6061786404ddb248d978d27302d800bee87ee
          Log:
          Merge pull request #1668 from jenkinsci/dont-send-excessive-classes

          JENKINS-28058 Don't send a reference to Computer.class over remoting channels

          Compare: https://github.com/jenkinsci/jenkins/compare/463b3fff2b86...3bf606178640

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/java/hudson/model/Computer.java http://jenkins-ci.org/commit/jenkins/3bf6061786404ddb248d978d27302d800bee87ee Log: Merge pull request #1668 from jenkinsci/dont-send-excessive-classes JENKINS-28058 Don't send a reference to Computer.class over remoting channels Compare: https://github.com/jenkinsci/jenkins/compare/463b3fff2b86...3bf606178640

          dogfood added a comment -

          Integrated in jenkins_main_trunk #4106
          [FIXED JENKINS-28058] Don't send a reference to Computer.class over remoting channels (Revision df831325b6623c479dcf4b215415f4523d10530b)

          Result = SUCCESS
          stephen connolly : df831325b6623c479dcf4b215415f4523d10530b
          Files :

          • core/src/main/java/hudson/model/Computer.java

          dogfood added a comment - Integrated in jenkins_main_trunk #4106 [FIXED JENKINS-28058] Don't send a reference to Computer.class over remoting channels (Revision df831325b6623c479dcf4b215415f4523d10530b) Result = SUCCESS stephen connolly : df831325b6623c479dcf4b215415f4523d10530b Files : core/src/main/java/hudson/model/Computer.java

          Daniel Beck added a comment -

          stephenconnolly Improvements are not really eligible for backporting. Can this be considered a bug?

          Daniel Beck added a comment - stephenconnolly Improvements are not really eligible for backporting. Can this be considered a bug?

          Stephen Connolly added a comment -

          https://github.com/jenkinsci/jenkins/commit/df831325b6623c479dcf4b215415f4523d10530b#diff-9fafdcd0712c5a5dab3acb4ea168515aR1197 fixes the actual bug. The other lines are improvements but do not actually change any API contracts.

          IMHO the sending over of Jenkins to remote instances is actually a bug

          Stephen Connolly added a comment - https://github.com/jenkinsci/jenkins/commit/df831325b6623c479dcf4b215415f4523d10530b#diff-9fafdcd0712c5a5dab3acb4ea168515aR1197 fixes the actual bug. The other lines are improvements but do not actually change any API contracts. IMHO the sending over of Jenkins to remote instances is actually a bug

          Daniel Beck added a comment -

          So let's just call this a bug and see what happens.

          Daniel Beck added a comment - So let's just call this a bug and see what happens.

          Daniel Beck added a comment -

          Rejected for 1.609.2 inclusion during the LTS RC status check today.

          Daniel Beck added a comment - Rejected for 1.609.2 inclusion during the LTS RC status check today.

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          core/src/main/java/hudson/model/Computer.java
          http://jenkins-ci.org/commit/jenkins/b591a241b4a437863bf48ebbd6aa4d3f2c0f831c
          Log:
          [FIXED JENKINS-28058] Don't send a reference to Computer.class over remoting channels

          • Logging added to help diagnose was actually causing Jenkins.class to be initialized on the remoting side

          (cherry picked from commit df831325b6623c479dcf4b215415f4523d10530b)

          Compare: https://github.com/jenkinsci/jenkins/compare/3005532a9456...b591a241b4a4

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/java/hudson/model/Computer.java http://jenkins-ci.org/commit/jenkins/b591a241b4a437863bf48ebbd6aa4d3f2c0f831c Log: [FIXED JENKINS-28058] Don't send a reference to Computer.class over remoting channels Logging added to help diagnose was actually causing Jenkins.class to be initialized on the remoting side (cherry picked from commit df831325b6623c479dcf4b215415f4523d10530b) Compare: https://github.com/jenkinsci/jenkins/compare/3005532a9456...b591a241b4a4

          dogfood added a comment -

          Integrated in jenkins_main_trunk #4292
          [FIXED JENKINS-28058] Don't send a reference to Computer.class over remoting channels (Revision b591a241b4a437863bf48ebbd6aa4d3f2c0f831c)

          Result = UNSTABLE
          ogondza : b591a241b4a437863bf48ebbd6aa4d3f2c0f831c
          Files :

          • core/src/main/java/hudson/model/Computer.java

          dogfood added a comment - Integrated in jenkins_main_trunk #4292 [FIXED JENKINS-28058] Don't send a reference to Computer.class over remoting channels (Revision b591a241b4a437863bf48ebbd6aa4d3f2c0f831c) Result = UNSTABLE ogondza : b591a241b4a437863bf48ebbd6aa4d3f2c0f831c Files : core/src/main/java/hudson/model/Computer.java

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

              Created:
              Updated:
              Resolved: