Story #16068
closed
[Process view] Process panel main view data
100%
Description
Merge the "Process info" and "process details" panels.
Use "ProcessDetailsAttributes" for both the right hand details and the info panel.
Update ProcessDetailsAttributes to include the following fields:
- Name
- UUID
- Owner UUID (ResourceOwnerWithName)
- Container status (queued, running, complete, failed) ("Locked" state should be rendered as "queued") this should continue to be rendered as a badge with color like it is now
- User it ran as (this is "runtime_user_uuid" of the underlying container object) (ResourceOwnerWithName)
- Create time
- Start time
- End time
- Run time (or "now" if not done yet)
- Link to output collection
- Link to log collection
- Link to docker image collection
- Link to "requesting_container_uuid" (if non-null)
- Properties
Things that should be displayed in separate, additional panels (will be separate stories):
- Logs
- Input collections
- Command line
- Runtime constraints / scheduling constraints / output storage class
(note: we might consider refactoring some of the fields currently listed in "details" into one of these other panels, but they have to be implemented first).
Related issues
Updated by Lucas Di Pentima over 2 years ago
- Related to Story #15333: Workbench2 feature parity with Workbench added
Updated by Lucas Di Pentima over 2 years ago
- Subject changed from Run summary (start/stop time) to Process panel detailed information data
- Run summary (start/stop time)
- who it ran as
- container uuid (it is being displayed as text)
- runtime constraints (enhance display style?)
- output_path is being displayed instead of output collection?
Updated by Lucas Di Pentima over 2 years ago
- Description updated (diff)
- Subject changed from Process panel detailed information data to Process panel main view data
Updated by Peter Amstutz over 2 years ago
- Target version deleted (
To Be Groomed)
Updated by Peter Amstutz over 2 years ago
- Category changed from Workbench2 to 0
- Subject changed from Process panel main view data to [Process view] Process panel main view data
Updated by Peter Amstutz over 1 year ago
- Related to Story #16248: Propose redesign of process view based on feedback added
Updated by Peter Amstutz over 1 year ago
- Related to Story #16945: WB2 Workflows / containers feature parity added
Updated by Peter Amstutz about 1 year ago
- Related to deleted (Story #15333: Workbench2 feature parity with Workbench)
Updated by Peter Amstutz 4 months ago
- Target version set to 2022-03-16 sprint
- Assigned To set to Stephen Smith
- Category deleted (
Workbench2) - Description updated (diff)
- Project changed from Arvados Workbench 2 to Arvados
Updated by Peter Amstutz 4 months ago
- Target version changed from 2022-03-16 sprint to 2022-03-30 Sprint
Updated by Peter Amstutz 3 months ago
- Target version changed from 2022-03-30 Sprint to 2022-04-13 Sprint
Updated by Stephen Smith 3 months ago
Changes at arvados-workbench2|444c41ca81c90ff172c7b1d99ff78f52bff991e8 branch 16068-merge-process-info-details-panels
Tests developer-tests-workbench2: #659
- Merged the 2 panels contents, removed info panel
- Added cypress waitForDom helper to reduce test flakiness
Updated by Stephen Smith 3 months ago
Updated at arvados-workbench2|5a0db55c7b4751b18f8d2a1ffe3ab3b5813b20c6
Tests developer-tests-workbench2: #663
- Merged main and added runtime status
Updated by Peter Amstutz 3 months ago
- Release changed from 47 to 51
- Target version changed from 2022-04-13 Sprint to 2022-04-27 Sprint
Updated by Lucas Di Pentima 3 months ago
Some initial comments:
- The process' status already is shown in the main UI as a badge.
- "Properties", "Requesting Container UUID" and "Run time" are missing.
- The "Type" field seems superfluous on the main process UI.
- I think we should skip the "Runtime constraints" data as it'll be implemented in a separate story (as per this story's comment).
- For UI consistency's sake, I think we should try to match the properties' ordering as much as possible with what we have on the collections UI: For example: The resource's and owner's UUIDs could be at the very top.
- Please check that the "Expires At" field is really used, if what's on our doc page (https://doc.arvados.org/v2.4/api/methods/container_requests.html) is correct, it isn't implemented yet so we can skip it.
- The "Outputs: /var/spool/cwl" field doesn't seem useful. Could you instead place the output collection's link as its value?
- The "Last modified" field seems useless to me, and it's not listed as a requirement nor used in wb1, so I think we could skip it. (Please double check with Peter)
As commented on video chat: Type and Status may not be rendered on the main Process' UI but should when used on the details right-hand side panel, so we may need some kind of switch to turn those on/off.
Updated by Stephen Smith 2 months ago
Updates at arvados-workbench2|1bb147efcf6a17c0b4237c790757dd54994005b2
tests developer-tests-workbench2: #669 /console
- Made type and status only show on process panel
- Added properties, requesting container uuid, runtime
- Removed constraints, expires, last modified, and extraneous outputs field
- Reordered fields to be similar to collections
- added
controllerURL=`echo "$controllerInfo" | awk '{print $1;}'`;
to integration script to get only first token of arvboot output (https://localhost:35219/ zzzzz localhost:35219)
Updated by Lucas Di Pentima 2 months ago
This is looking great! Just one remaining observation:
- On the right-side details panel, the displayed "Status" attribute should be the same as the status badge on the main UI instead of the container's request "State" field. (the process status is derived from both container request & container states -- you can use
getProcessStatus()to get it)
The rest LGTM, thanks!
Updated by Stephen Smith 2 months ago
- % Done changed from 0 to 100
- Status changed from In Progress to Resolved
Applied in changeset arvados-workbench-2:arvados-workbench2|2742a401dcc330697a14d2edb3c4c81115f67402.