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

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

    XMLWordPrintable

Details

    • Bug
    • Status: Open (View Workflow)
    • Minor
    • Resolution: Unresolved
    • gerrit-trigger-plugin
    • None
    • Jenkins 2.89.2
      Gerrit Trigger plugin 2.27.2

    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

          Descending order - Click to sort in ascending order
          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

          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
          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

          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

          People

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

            Dates

              Created:
              Updated: