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

Show more log is not working properly

    XMLWordPrintable

    Details

    • Sprint:
      Blue Ocean 1.1
    • Similar Issues:

      Description

      Context
      Instead of fixing this and potentially opening a can of worms with our log component we are going to degrade functionality here for the time being until we schedule the "Log Improvements" epic

      Scope

      • Clicking "Show complete log" opens the log for the step in a new window
        • Functionally the same as the "open log for stage/parallel" feature pictured below

      Original request
      https://media.giphy.com/media/l0Iy8VknYtOxjmQQU/giphy.gif
      What is should do: show all the log for the step inline? (not sure it has even been specced). 
      I expect this has been broken for some time (since the log logic changed, to my protests). 
      A ATH test should cover this (there was meant to already be one). 

        Attachments

          Issue Links

            Activity

            Hide
            michaelneale Michael Neale added a comment -

            Suggested fix from James Dumay:

             

            Just make the "show full log" open the log for the step in a new window, the same as the "pop out" button does (this is basically a hyperlink) to show ALL the log for the step as plain text. There will be a follow on ticket to make this work the "nicer" way in the Log improvements epic. 

             

            Show
            michaelneale Michael Neale added a comment - Suggested fix from James Dumay :   Just make the "show full log" open the log for the step in a new window, the same as the "pop out" button does (this is basically a hyperlink) to show ALL the log for the step as plain text. There will be a follow on ticket to make this work the "nicer" way in the Log improvements epic.   
            Hide
            cliffmeyers Cliff Meyers added a comment -

            I've implemented a fix based on James Dumay suggestion. PR is linked.

            Show
            cliffmeyers Cliff Meyers added a comment - I've implemented a fix based on James Dumay suggestion. PR is linked.
            Hide
            michaelneale Michael Neale added a comment -

            thanks cliff!

            Show
            michaelneale Michael Neale added a comment - thanks cliff!

              People

              • Assignee:
                cliffmeyers Cliff Meyers
                Reporter:
                imeredith Ivan Meredith
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: