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

display name can no longer be used as graph label

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • robot-plugin
    • Robot Framework plugin 2.1.2
    • 3.5.1

      Before the PR from JENKINS-35327, the build numbers we set using plugin Set Build Number was shown on the graph, after the PR was merged the trend graph always shows the internal build number and not our customized build number.

          [JENKINS-63900] display name can no longer be used as graph label

          Aleksi Simell added a comment -

          New version released (it will take up to 8 hours to show in update-center).

          Aleksi Simell added a comment - New version released (it will take up to 8 hours to show in update-center).

          Vitor Rego added a comment - - edited

          Hi aleksisimell, I´m not sure if version 3.0.0 might fix the issue, but still not fixed:

          If I set the variable I want in the global settings, the graph does not show anything, even after I clean by browser´s cache:

           

          Vitor Rego added a comment - - edited Hi aleksisimell , I´m not sure if version 3.0.0 might fix the issue, but still not fixed: If I set the variable I want in the global settings, the graph does not show anything, even after I clean by browser´s cache:  

          Aleksi Simell added a comment -

          vitorego you can specify the label per job now. Try that.

          Aleksi Simell added a comment - vitorego you can specify the label per job now. Try that.

          Aleksi Simell added a comment -

          vitorego Did configuring the label for the job resolve your issue?

          Aleksi Simell added a comment - vitorego Did configuring the label for the job resolve your issue?

          Vitor Rego added a comment -

          Hi aleksisimell, using the setting per job the graph is updated, however all builds in the graph are updated with the latest build:

          Vitor Rego added a comment - Hi aleksisimell , using the setting per job the graph is updated, however all builds in the graph are updated with the latest build:

          Aleksi Simell added a comment - - edited

          I'm actually genuinely surprised that has even worked in the first place. Digging back to earlier PR it used NumberOnlyBuildLabel, which is supposed to return integers only.

          Aleksi Simell added a comment - - edited I'm actually genuinely surprised that has even worked in the first place. Digging back to earlier PR it used NumberOnlyBuildLabel , which is supposed to return integers only.

          Aleksi Simell added a comment -

          Reopening the issue again, as this is clearly not fixed yet.

          Aleksi Simell added a comment - Reopening the issue again, as this is clearly not fixed yet.

          Vitor Rego added a comment -

          The graph now is showing the build number, but the same buid number (the last one) is used for all the runs:

          Vitor Rego added a comment - The graph now is showing the build number, but the same buid number (the last one) is used for all the runs:

          Aleksi Simell added a comment -

          Finally managed to fix this. Pull Request created to fix this issue. The root cause is that you don't actually want to use build number as the label, but rather the build display name. This PR adds possibility to use display name as the label.

          Aleksi Simell added a comment - Finally managed to fix this. Pull Request created to fix this issue. The root cause is that you don't actually want to use build number as the label, but rather the build display name . This PR adds possibility to use display name as the label.

          Aleksi Simell added a comment -

          PR merged. Fix will be available in the next release.

          Aleksi Simell added a comment - PR merged. Fix will be available in the next release.

            aleksisimell Aleksi Simell
            vitorego Vitor Rego
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: