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

links to robot framework report files are invalid when inside a folder

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Links to robot framework report files are invalid (404) when the job is inside a folder (from CloudBees Folder plugin).
      For example, instead of:

       /job/my_folder/my_job/3446/robot/report/report.html

      ... I should get:

       /job/my_folder/job/my_job/3446/robot/report/report.html

        Attachments

          Activity

          Hide
          rdesgroppes Régis Desgroppes added a comment -

          Very simple fix working for all kinds of jobs, given the Run.getUrl() provides the right path we need.
          Verified with trivial job, matrix axis, item within a folder.
          All tests pass.
          Could you please consider applying it to the main branch?

          Show
          rdesgroppes Régis Desgroppes added a comment - Very simple fix working for all kinds of jobs, given the Run.getUrl() provides the right path we need. Verified with trivial job, matrix axis, item within a folder. All tests pass. Could you please consider applying it to the main branch?
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Regis Desgroppes
          Path:
          src/main/resources/hudson/plugins/robot/util/robotsummary.jelly
          http://jenkins-ci.org/commit/robot-plugin/6c490c5a40fe1c2823a40a21dea6d533c0143dde
          Log:
          [FIXED JENKINS-21739] links to Robot Framework files are now valid for jobs residing inside folders (i.e. CloudBees Folders plugin). Also verified for trivial jobs and matrix axis.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Regis Desgroppes Path: src/main/resources/hudson/plugins/robot/util/robotsummary.jelly http://jenkins-ci.org/commit/robot-plugin/6c490c5a40fe1c2823a40a21dea6d533c0143dde Log: [FIXED JENKINS-21739] links to Robot Framework files are now valid for jobs residing inside folders (i.e. CloudBees Folders plugin). Also verified for trivial jobs and matrix axis.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Régis Desgroppes
          Path:
          src/main/resources/hudson/plugins/robot/util/robotsummary.jelly
          http://jenkins-ci.org/commit/robot-plugin/2557f2e176499f521d032b42dbe9ad3e0bce5296
          Log:
          Merge pull request #6 from rdesgroppes/master

          [FIXED JENKINS-21739] links to Robot Framework files are now valid for jobs residing inside folders

          Compare: https://github.com/jenkinsci/robot-plugin/compare/bbc4eb96f939...2557f2e17649

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Régis Desgroppes Path: src/main/resources/hudson/plugins/robot/util/robotsummary.jelly http://jenkins-ci.org/commit/robot-plugin/2557f2e176499f521d032b42dbe9ad3e0bce5296 Log: Merge pull request #6 from rdesgroppes/master [FIXED JENKINS-21739] links to Robot Framework files are now valid for jobs residing inside folders Compare: https://github.com/jenkinsci/robot-plugin/compare/bbc4eb96f939...2557f2e17649
          Hide
          ninian Ninian added a comment -

          Any chance of getting a release produced with this fix in it? It would be a big help for us, and it looks a fairly low risk change for other users.

          Show
          ninian Ninian added a comment - Any chance of getting a release produced with this fix in it? It would be a big help for us, and it looks a fairly low risk change for other users.
          Hide
          jpiironen jpiironen added a comment -

          Sorry for the late reply. The fix by Régis is now merged to the master branch. Thank you so much for the contribution! We recently started working towards a new release, which should be expected soon.

          Show
          jpiironen jpiironen added a comment - Sorry for the late reply. The fix by Régis is now merged to the master branch. Thank you so much for the contribution! We recently started working towards a new release, which should be expected soon.

            People

            • Assignee:
              jpiironen jpiironen
              Reporter:
              rdesgroppes Régis Desgroppes
            • Votes:
              1 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: