JenkinsIcon indicating the project type

Jenkins

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

Excessive classes being sent to slave machine

    Details

    • Similar Issues:

      Description

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

        Attachments

          Activity

          Descending order - Click to sort in ascending order
          Hide
          Permalink
          dogfood 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
          Show
          dogfood 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
          Hide
          Permalink
          scm_issue_link 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

          Show
          scm_issue_link 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
          Hide
          Permalink
          danielbeck Daniel Beck added a comment -

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

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

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

          Show
          danielbeck Daniel Beck added a comment - So let's just call this a bug and see what happens.
          Hide
          Permalink
          stephenconnolly 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

          Show
          stephenconnolly 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
          4 older comments

            People

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

              Dates

              • Created:
                Updated:
                Resolved: