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

Link to the console output missing in popup when log >200Kb

    XMLWordPrintable

Details

    Description

      When looking into the popup in the breadcrumb of the lastBuild of a particular job, there is a link to the console of the last build (GOOD). See for example, the popup of the breadcrumb in https://buildhive.cloudbees.com/job/cgeo/job/c-geo-opensource/lastBuild/

      And there is also a link to the lastBuild console in the popup of the lastBuild permalink in this job page (GOOD).
      See for example, the "good.png" attached screenshot.

      But in some other job (log file larger than 200000 probably), there is no link to the console of the last build in the breadcrumb of the lastBuild page (BAD).
      See for example, the popup of the breadcrumb in https://buildhive.cloudbees.com/job/Atmosphere/job/atmosphere/lastBuild/

      And in the job page of the same job, there is no link to the lastBuild console in the popup of the lastBuild permalink in the same job page (BAD).
      See for exampe, the "BAD.png" attached screenshot.

      Attachments

        1. BAD.png
          BAD.png
          90 kB
        2. good.png
          good.png
          85 kB

        Issue Links

          Activity

            dogfood dogfood added a comment -

            Integrated in jenkins_main_trunk #3473
            [FIXED JENKINS-14264] For logs >200Kb, just show two <t:task>s without nesting, for proper interoperability with the context menu. (Revision 87e2d32faa4f4de9793c7cfcdc365baceedb0411)
            JENKINS-14264 Factor out console-link.jelly for ease of reuse from other run types with logs. (Revision 9404759d0d76951372688c02100fbfc5f383ff6d)

            Result = SUCCESS
            Jesse Glick : 87e2d32faa4f4de9793c7cfcdc365baceedb0411
            Files :

            • core/src/main/resources/hudson/model/AbstractBuild/tasks.jelly
            • changelog.html

            Jesse Glick : 9404759d0d76951372688c02100fbfc5f383ff6d
            Files :

            • core/src/main/resources/lib/hudson/project/console-link.jelly
            • core/src/main/resources/hudson/model/AbstractBuild/tasks.jelly
            dogfood dogfood added a comment - Integrated in jenkins_main_trunk #3473 [FIXED JENKINS-14264] For logs >200Kb, just show two <t:task>s without nesting, for proper interoperability with the context menu. (Revision 87e2d32faa4f4de9793c7cfcdc365baceedb0411) JENKINS-14264 Factor out console-link.jelly for ease of reuse from other run types with logs. (Revision 9404759d0d76951372688c02100fbfc5f383ff6d) Result = SUCCESS Jesse Glick : 87e2d32faa4f4de9793c7cfcdc365baceedb0411 Files : core/src/main/resources/hudson/model/AbstractBuild/tasks.jelly changelog.html Jesse Glick : 9404759d0d76951372688c02100fbfc5f383ff6d Files : core/src/main/resources/lib/hudson/project/console-link.jelly core/src/main/resources/hudson/model/AbstractBuild/tasks.jelly
            jglick Jesse Glick added a comment - - edited

            Assigning back to myself, not sure why @danielbeck took it.

            jglick Jesse Glick added a comment - - edited Assigning back to myself, not sure why @danielbeck took it.
            danielbeck Daniel Beck added a comment -

            No idea why I did that, possibly because I had an outdated web page open. Sorry.

            danielbeck Daniel Beck added a comment - No idea why I did that, possibly because I had an outdated web page open. Sorry.

            Code changed in jenkins
            User: Jesse Glick
            Path:
            core/src/main/resources/hudson/model/AbstractBuild/tasks.jelly
            http://jenkins-ci.org/commit/jenkins/001e51e87ceff6d02b8dbaf4d632eac55d4de47d
            Log:
            [FIXED JENKINS-14264] For logs >200Kb, just show two <t:task>s without nesting, for proper interoperability with the context menu.
            (cherry picked from commit 87e2d32faa4f4de9793c7cfcdc365baceedb0411)

            Conflicts:
            changelog.html

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: core/src/main/resources/hudson/model/AbstractBuild/tasks.jelly http://jenkins-ci.org/commit/jenkins/001e51e87ceff6d02b8dbaf4d632eac55d4de47d Log: [FIXED JENKINS-14264] For logs >200Kb, just show two <t:task>s without nesting, for proper interoperability with the context menu. (cherry picked from commit 87e2d32faa4f4de9793c7cfcdc365baceedb0411) Conflicts: changelog.html

            Code changed in jenkins
            User: Jesse Glick
            Path:
            core/src/main/resources/hudson/model/AbstractBuild/tasks.jelly
            core/src/main/resources/lib/hudson/project/console-link.jelly
            http://jenkins-ci.org/commit/jenkins/8f576cda1d7dcf3f71803e2fe2f5eb4b3f3340df
            Log:
            JENKINS-14264 Factor out console-link.jelly for ease of reuse from other run types with logs.
            (cherry picked from commit 9404759d0d76951372688c02100fbfc5f383ff6d)

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: core/src/main/resources/hudson/model/AbstractBuild/tasks.jelly core/src/main/resources/lib/hudson/project/console-link.jelly http://jenkins-ci.org/commit/jenkins/8f576cda1d7dcf3f71803e2fe2f5eb4b3f3340df Log: JENKINS-14264 Factor out console-link.jelly for ease of reuse from other run types with logs. (cherry picked from commit 9404759d0d76951372688c02100fbfc5f383ff6d)

            People

              jglick Jesse Glick
              evernat evernat
              Votes:
              6 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: