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

Gerrit Trigger - Diagnostic Events Listeners - Links to jobs located in Folders are incorrect

    Details

    • Type: Bug
    • Status: Open (View Workflow)
    • Priority: Minor
    • Resolution: Unresolved
    • Component/s: gerrit-trigger-plugin
    • Labels:
      None
    • Environment:
      Jenkins 2.89.2
      Gerrit Trigger plugin 2.27.2
    • Similar Issues:

      Description

      In the Diagnostic Events Listeners page, links to jobs located in Folders are not generated properly.

       

      Scenario

      Input: job name gerrit_trigger in folder NEW_FOLDER. proper Url is <jenkins_url>/job/NEW_FOLDER/job/gerrit_trigger/

      Use Case: Go to <jenkins_url>/gerrit-trigger/diagnostics/eventListeners/

      Expected: displayed job gerrit_trigger has a link to <jenkins_url>/job/NEW_FOLDER/job/gerrit_trigger/

      Actual: displayed job gerrit_trigger has a link to <jenkins_url>/job/gerrit_trigger/ --> not working

       

      Proposed solution:

      use getUrl() instead of getShortUrl() in https://github.com/jenkinsci/gerrit-trigger-plugin/blob/master/src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/diagnostics/EventListenersReport/index.groovy#L101**

        Attachments

          Activity

          geoffroyjabouley Geoffroy Jabouley created issue -
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Geoffroy Jabouley
          Path:
          src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/diagnostics/EventListenersReport/index.groovy
          http://jenkins-ci.org/commit/gerrit-trigger-plugin/adf1b2b6cd992c5f25a6a2d97fc4275bd9871866
          Log:
          JENKINS-49163 Create correct url in Diagnotic Event Listeners report for jobs located in folders

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Geoffroy Jabouley Path: src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/diagnostics/EventListenersReport/index.groovy http://jenkins-ci.org/commit/gerrit-trigger-plugin/adf1b2b6cd992c5f25a6a2d97fc4275bd9871866 Log: JENKINS-49163 Create correct url in Diagnotic Event Listeners report for jobs located in folders
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Robert Sandell
          Path:
          src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/diagnostics/EventListenersReport/index.groovy
          http://jenkins-ci.org/commit/gerrit-trigger-plugin/cba711247812b28610f904b537f1f088250faef6
          Log:
          Merge pull request #348 from geoffroyjabouley/master

          JENKINS-49163 Create correct url in Diagnotic Event Listeners report for jobs located in folders

          Compare: https://github.com/jenkinsci/gerrit-trigger-plugin/compare/9f4cf73aa45b...cba711247812

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Robert Sandell Path: src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/diagnostics/EventListenersReport/index.groovy http://jenkins-ci.org/commit/gerrit-trigger-plugin/cba711247812b28610f904b537f1f088250faef6 Log: Merge pull request #348 from geoffroyjabouley/master JENKINS-49163 Create correct url in Diagnotic Event Listeners report for jobs located in folders Compare: https://github.com/jenkinsci/gerrit-trigger-plugin/compare/9f4cf73aa45b...cba711247812

            People

            • Assignee:
              rsandell rsandell
              Reporter:
              geoffroyjabouley Geoffroy Jabouley
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated: