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

Fix showing custom computer icon in executor status window

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • None

      Fix showing custom computer icon in executor status window. If somebody creates a plugin with a custom Computer type and overrides the getIcon() and getIconClassName() methods, the new icon will be displayed in the /computer/ page of Jenkins. However, the executor status window has the default icon hard-coded into it and so it will not be updated.

      See: https://github.com/jenkinsci/jenkins/blob/master/core/src/main/resources/lib/hudson/executors.jelly#L37
      https://github.com/jenkinsci/jenkins/blob/master/core/src/main/java/hudson/model/Computer.java#L730

          [JENKINS-36775] Fix showing custom computer icon in executor status window

          Sebastien Guillemot created issue -
          Sebastien Guillemot made changes -
          Description Original: Fix showing custom computer icon in executor status window. If somebody creates a plugin with a custom Computer type and overrides the getIcon() and getIconClassName() methods, the new icon will be displayed in the /computer/ page of Jenkins. However, the executor status window has the default icon hard-coded into it and so it will not be updated.

          See: https://github.com/jenkinsci/jenkins/blob/c528cdc7ff3cf7473008f0ede811c65c2a48d132/core/src/main/resources/lib/hudson/executors.jelly#L37
          https://github.com/jenkinsci/jenkins/blob/master/core/src/main/java/hudson/model/Computer.java#L730
          New: Fix showing custom computer icon in executor status window. If somebody creates a plugin with a custom Computer type and overrides the getIcon() and getIconClassName() methods, the new icon will be displayed in the /computer/ page of Jenkins. However, the executor status window has the default icon hard-coded into it and so it will not be updated.

          See: https://github.com/jenkinsci/jenkins/blob/master/core/src/main/resources/lib/hudson/executors.jelly#L37
          https://github.com/jenkinsci/jenkins/blob/master/core/src/main/java/hudson/model/Computer.java#L730

          Daniel Beck added a comment -

          Fixed in 2.15

          Daniel Beck added a comment - Fixed in 2.15
          Daniel Beck made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 173512 ] New: JNJira + In-Review [ 199468 ]

            segu Sebastien Guillemot
            segu Sebastien Guillemot
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: