Skip to content

Conversation

@dtaylor113
Copy link
Contributor

JIRA: https://jira.coreos.com/browse/CONSOLE-1050
"If I pause a log with long lines, then try to scroll horizontally, the log stream resumes."

Fixed so that horizontal scroll does NOT resume paused Log.

@dtaylor113 dtaylor113 requested a review from spadgett January 29, 2019 19:57
@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 29, 2019
@dtaylor113 dtaylor113 requested a review from TheRealJon January 30, 2019 15:58
Copy link
Contributor

@alecmerdler alecmerdler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!
/lgtm

@dtaylor113
Copy link
Contributor Author

Hi @spadgett and @TheRealJon, pls review when you get a moment -thanks

@spadgett
Copy link
Member

spadgett commented Feb 1, 2019

/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 1, 2019
@openshift openshift deleted a comment from openshift-ci-robot Feb 4, 2019
@dtaylor113
Copy link
Contributor Author

/retest

@spadgett
Copy link
Member

spadgett commented Feb 4, 2019

@dtaylor113 fyi, there's still an outstanding review comment

@dtaylor113
Copy link
Contributor Author

@dtaylor113 fyi, there's still an outstanding review comment

@spadgett, addressed -thanks

Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 4, 2019
Copy link
Member

@TheRealJon TheRealJon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@TheRealJon
Copy link
Member

/retest

@spadgett
Copy link
Member

spadgett commented Feb 4, 2019

/hold cancel
/retest

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 4, 2019
@spadgett
Copy link
Member

spadgett commented Feb 5, 2019

/retest

5 similar comments
@spadgett
Copy link
Member

spadgett commented Feb 5, 2019

/retest

@dtaylor113
Copy link
Contributor Author

/retest

@dtaylor113
Copy link
Contributor Author

/retest

@openshift-merge-robot
Copy link
Contributor

/retest

@openshift-merge-robot
Copy link
Contributor

/retest

@openshift-merge-robot openshift-merge-robot merged commit 941b569 into openshift:master Feb 6, 2019
@dtaylor113 dtaylor113 deleted the log-pause branch August 22, 2019 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants