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

TAP Plugin generates bad detail links on "tapTestReport" page

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      (See this page for the original report of this bug in context)

      The fix for JENKINS-16971 (e8dcf5c) breaks my Jenkins TAP plugin installation. Specifically, the detail links on the http://jenkins.example.org/job/JOBNAME/BUILDNUM/tapTestReport/ page are incorrect. They should read something like http://jenkins.example.org/job/JOBNAME/BUILDNUM/tapTestReport/summary.tap-1/. Instead they read something like http://jenkins.example.org/var/lib/jenkins/jobs/JOBNAME/builds/2013-04-04_12-43-45/tap-master-files/test/mpi/summary.tap-1/. Obviously this URL is invalid and yields an HTTP 404 from the web server.
      I suspect the problem is caused by the replacement of getName() with getAbsolutePath(). Reverting commit e8dcf5c fixes the error in my otherwise clean build of 80a01b1. I made no attempt to test or understand the issue actually reported in and/or addressed by JENKINS-16971.

        Attachments

          Activity

          Hide
          kinow Bruno P. Kinoshita added a comment -

          @Ken, weird. I'll take a look tomorrow Ken. Sorry for that :-/

          Show
          kinow Bruno P. Kinoshita added a comment - @Ken, weird. I'll take a look tomorrow Ken. Sorry for that :-/
          Hide
          kinow Bruno P. Kinoshita added a comment -

          Reopening due to regression

          Show
          kinow Bruno P. Kinoshita added a comment - Reopening due to regression
          Hide
          raffenet Ken Raffenetti added a comment -

          No problem. Let me know if you need any more info. I've gone back to my custom 1.11 snapshot with the troublesome commit Dave cited git revert'ed out.

          Show
          raffenet Ken Raffenetti added a comment - No problem. Let me know if you need any more info. I've gone back to my custom 1.11 snapshot with the troublesome commit Dave cited git revert'ed out.
          Hide
          kinow Bruno P. Kinoshita added a comment -

          Fixed on issues 18885 and 17855, with commit http://jenkins-ci.org/commit/tap-plugin/21e7af07d5ae18327eb7a2626184970a6ff1d995

          Sorry for mixing issues in the same commit, but I started working on 18885 this morning and it included this issue too.

          Cutting a new release tonight, tomorrow morning/noon you should be able to give 1.12 a try.

          Thanks!

          Show
          kinow Bruno P. Kinoshita added a comment - Fixed on issues 18885 and 17855, with commit http://jenkins-ci.org/commit/tap-plugin/21e7af07d5ae18327eb7a2626184970a6ff1d995 Sorry for mixing issues in the same commit, but I started working on 18885 this morning and it included this issue too. Cutting a new release tonight, tomorrow morning/noon you should be able to give 1.12 a try. Thanks!
          Hide
          kinow Bruno P. Kinoshita added a comment -

          Fixed in 1.12

          Show
          kinow Bruno P. Kinoshita added a comment - Fixed in 1.12

            People

            • Assignee:
              kinow Bruno P. Kinoshita
              Reporter:
              goodell Dave Goodell
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: