Feature #18881
closed
Display container runtime_status
Added by Peter Amstutz about 2 years ago.
Updated almost 2 years ago.
Release relationship:
Auto
Description
runtime status can have "warning" and "error" fields.
If there's an "error" the container should be displayed as in a "failing" state.
When the process appears in the project list or "all projects" or "child processes" it should show an icon when it is in "failing" state.
On the process main panel it should show all the warnings and errors.
Files
- Related to Idea #16945: WB2 Workflows / containers feature parity added
- Description updated (diff)
- Status changed from New to In Progress
- Target version changed from 2022-03-30 Sprint to 2022-04-13 Sprint
Updates at arvados-workbench2|a5750c2 - branch 18881-process-runtime-status
Test run: developer-tests-workbench2: #649 
- Adds "Failing" and "Warning" as new process states that depend not only on the container's state but also the runtime status data.
- Fixes status indications by conforming on how workbech1 renders them.
- Adds runtime status indications to the process info panel.
- Adds tests.
Lucas Di Pentima wrote:
Updates at arvados-workbench2|078bedef - developer-tests-workbench2: #652 
- Adds workarounds on unrelated flaky tests.
This looks great! My only comment is that it would be nice to use the same colorful button-style status fields in the process list view (e.g. "All Processes"), rather than the current colored text there, some of which can be hard to read (e.g. 'Queued' in light gray on a white background). I think it would also be nicer aesthetically.
LGTM otherwise.
Lucas Di Pentima wrote:
Updates at arvados-workbench2|18425d0d - developer-tests-workbench2: #653 
- Makes the process status indicator to be consistent on listings with the style that's used on the process info panel.
- Updates tests.
Loving the buttons in the data tables! But I guess something went wrong in the project listing, cf.
- % Done changed from 0 to 100
- Status changed from In Progress to Resolved
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by 