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

monitoring-plugin description in manage jenkins looks wrong

    XMLWordPrintable

Details

    Description

      Monitoring of Jenkins master

Monitoring of memory, cpu, http requests and more in Jenkins master.
You can also view the monitoring of

Jenkins nodes

.

      I believe the description comes from:

      https://github.com/jenkinsci/monitoring-plugin/blob/aafbd8ce7c303bd0711117d983fb0d7364168776/src/main/java/org/jvnet/hudson/plugins/monitoring/PluginManagementLink.java#L54-L55

      And I think the rendering comes from:

      https://github.com/jenkinsci/jenkins/blob/4d65572c6db9099ebcf8d2222159c069996670f9/core/src/main/resources/jenkins/model/Jenkins/manage.jelly#L51-L66

      Offhand, the simplest change would be to remove the . from the end of the description. That isn't ideal, but... it'd be better than the current output.

      Attachments

        Activity

          Ascending order - Click to sort in descending order
          evernat evernat added a comment - - edited

          Yes, I know this issue. Thanks for reminding it to me.

          With the following description in PluginManagementLink, it is better now:

          "Monitoring of memory, cpu, http requests and more in Jenkins master."
          + "<a href='./monitoring/nodes'>You can also view the monitoring of builds, build queue and Jenkins nodes.</a>"

          See this screenshot:

          It is not immediately obvious but the two first lines have a link to the "/monitoring" page.

          And the third line has a link to the "/monitoring/nodes" page.

          evernat evernat added a comment - - edited Yes, I know this issue. Thanks for reminding it to me. With the following description in PluginManagementLink, it is better now: "Monitoring of memory, cpu, http requests and more in Jenkins master." + "<a href='./monitoring/nodes'>You can also view the monitoring of builds, build queue and Jenkins nodes.</a>" See this screenshot: It is not immediately obvious but the two first lines have a link to the "/monitoring" page. And the third line has a link to the "/monitoring/nodes" page.
          evernat evernat added a comment -

          Fixed by https://github.com/jenkinsci/monitoring-plugin/commit/eb2086790605d11504287496c82b063cb17a2be8

          evernat evernat added a comment - Fixed by https://github.com/jenkinsci/monitoring-plugin/commit/eb2086790605d11504287496c82b063cb17a2be8
          evernat evernat added a comment -

          released in 1.78.0

          evernat evernat added a comment - released in 1.78.0

          People

            evernat evernat
            jsoref Josh Soref
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: