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

Karaoke does not resume after "scroll to bottom"

    Details

    • Similar Issues:
    • Sprint:
      Blue Ocean 1.1, Blue Ocean 1.1, Blue Ocean 1.2-beta1, Blue Ocean 1.4 - beta 2

      Description

      Original description
      When viewing the Run Details logs for a running build, when a user scrolls up, the system stops loading logs entirely. This obviously prevents any karaoke behavior from occurring. It appears that when the build completes, we do refresh the data and logs to reflect the completed state.

      I have tested this in the following commits and the behavior seems consistent:

      There seems to be some debate as to what the behavior should be. Thorsten Scherler indicated it's always worked this way, although James Dumay says that log loading should continue regardless, just no auto scrolling / auto stage moving.

      Before I dig deeper into JENKINS-43316 and JENKINS-43317 it'd help to know whether this issue needs addressing. And if it's true that karaoke has always worked like this, this should be marked as an enhancement and probably not block 1.1 release as it may take some time to implement this from scratch. I don't know the code base well enough to estimate the effort.

        Attachments

          Issue Links

            Activity

            Hide
            jamesdumay James Dumay added a comment -

            Cliff Meyers thats spot on. I'd also say that when karaoke is paused:

            • Clicking on another active step reactivates karaoke
            Show
            jamesdumay James Dumay added a comment - Cliff Meyers thats spot on. I'd also say that when karaoke is paused: Clicking on another active step reactivates karaoke
            Hide
            cliffmeyers Cliff Meyers added a comment -

            Josh McDonald btw I linked two tickets to this one, JENKINS-43316 and JENKINS-43317. I know Keith Zantow had been working on a significant refactor to Karaoke to resolve all these tickets, but I believe it still had some bugs in it. May want to touch base with him to see if that branch can be carried forward?

            Show
            cliffmeyers Cliff Meyers added a comment - Josh McDonald btw I linked two tickets to this one, JENKINS-43316 and JENKINS-43317 . I know Keith Zantow had been working on a significant refactor to Karaoke to resolve all these tickets, but I believe it still had some bugs in it. May want to touch base with him to see if that branch can be carried forward?
            Hide
            jamesdumay James Dumay added a comment -

            This no longer a problem

            Show
            jamesdumay James Dumay added a comment - This no longer a problem
            Hide
            christianciach Christian Ciach added a comment -

            "Do not auto scroll to bottom as new log data is appended" is still an issue for me. This is very irritating, because our logs are extremely long and every time I try to scroll up (for example using the Home-key on the keyboard), it immediately scrolls all the way down again. Is this a known bug?

            Show
            christianciach Christian Ciach added a comment - "Do not auto scroll to bottom as new log data is appended" is still an issue for me. This is very irritating, because our logs are extremely long and every time I try to scroll up (for example using the Home-key on the keyboard), it immediately scrolls all the way down again. Is this a known bug?
            Hide
            sophistifunk Josh McDonald added a comment -

            Christian Ciach work on addressing these issues is currently in progress.

            Show
            sophistifunk Josh McDonald added a comment - Christian Ciach work on addressing these issues is currently in progress.

              People

              • Assignee:
                sophistifunk Josh McDonald
                Reporter:
                cliffmeyers Cliff Meyers
              • Votes:
                3 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: