Feature #18975
closed
log viewer improvements
Description
I'm using it to monitor the 2.4 test run.
https://workbench2.pirca.arvadosapi.com/processes/pirca-xvhdp-aiai8cvkddk1c4o
It's pretty great. I particularly like that when I've just started a workflow, the log panel is empty, but as soon as it starts running the logs start showing up, this gives good feedback.
- bug: If you start from empty, and then it starts logging stuff, the "All logs" dropdown won't populate with the other log streams until you reload the panel
- It would be nice if it started at the bottom and followed scroll by default, or had a button to do that
- In fact, currently it doesn't seem to follow scrolling at all. It really needs to do that. I think the way wb1 does it is by checking if it is scrolled all the way to the bottom, and then when adding new text, it updates the scroll to ensure it is still scrolled to the bottom. (However if it is not scrolled all the way to the bottom, then it doesn't mess with the scroll position)
- An option to show only crunch-run, stdout and stderr would be really great. If it was default, even better.
- Can we make clickable links (PDHs and uuids) open in a new tab? It's annoying if I click on a link and when I come back to the panel I've lost all my UI state (like the scroll position).
- Clickable link color is too dark against the black background
Out of scope (should be moved to their own tickets):
- Logs for completed containers should be read from the log collection
- Text search would be really great
Updated by Lucas Di Pentima about 2 years ago
- Assigned To set to Lucas Di Pentima
Updated by Peter Amstutz almost 2 years ago
- Target version changed from 2022-04-27 Sprint to 2022-05-11 sprint
Updated by Peter Amstutz almost 2 years ago
- Target version changed from 2022-05-11 sprint to 2022-05-25 sprint
Updated by Peter Amstutz almost 2 years ago
- Target version changed from 2022-05-25 sprint to 2022-06-08 sprint
Updated by Peter Amstutz almost 2 years ago
- Target version changed from 2022-06-08 sprint to 2022-06-22 Sprint
Updated by Lucas Di Pentima almost 2 years ago
- Target version changed from 2022-06-22 Sprint to 2022-07-06
Updated by Lucas Di Pentima almost 2 years ago
- Status changed from New to In Progress
Updated by Lucas Di Pentima almost 2 years ago
- Target version changed from 2022-07-06 to 2022-07-20
Updated by Lucas Di Pentima almost 2 years ago
Updates at arvados-workbench2|9ae74b6d - branch 18975-log-improvements
Test run: developer-tests-workbench2: #771
- Improves link coloring by using light green for better contrast against the background.
- Adds a "Main log" filter (selected by default because it's the first) that only shows crunch-run, stdout and stderr streams.
- Makes new arriving event types populate the filter drop down list.
- Bonus: Adds "keepstore" stream to the list (just realized that those logs will be available when using a local keepstore)
- Enables "follow mode" by default.
- Fixes "follow mode" when the panel is maximized by adding some margin to the scrolling & height calculations.
- Opens a new tab when clicking linked (and recognizable) UUIDs and PDHs.
- Bonus: improves the word-wrapping indication by changing the button's icon & tooltip.
Updated by Lucas Di Pentima almost 2 years ago
Although Cypress tests are failing on Jenkins, I was able to run them without issues in docker on my local VM.
Updated by Peter Amstutz almost 2 years ago
Lucas Di Pentima wrote in #note-14:
Updates at arvados-workbench2|9ae74b6d - branch
18975-log-improvements
Test run: developer-tests-workbench2: #771
- Improves link coloring by using light green for better contrast against the background.
- Adds a "Main log" filter (selected by default because it's the first) that only shows crunch-run, stdout and stderr streams.
- Makes new arriving event types populate the filter drop down list.
- Bonus: Adds "keepstore" stream to the list (just realized that those logs will be available when using a local keepstore)
- Enables "follow mode" by default.
- Fixes "follow mode" when the panel is maximized by adding some margin to the scrolling & height calculations.
- Opens a new tab when clicking linked (and recognizable) UUIDs and PDHs.
- Bonus: improves the word-wrapping indication by changing the button's icon & tooltip.
I know you are still fighting with the testing. Functionally, this seems like a great improvement, thank you! LGTM
Updated by Lucas Di Pentima almost 2 years ago
I was able to get a successful test run by using a worker node with 8GB of RAM: developer-tests-workbench2: #805
Updated by Lucas Di Pentima almost 2 years ago
- Status changed from In Progress to Resolved
Applied in changeset arvados-workbench-2:arvados-workbench2|c492c89c16f1e5061da277e112aa06777de4461e.