Arvados

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 at arvados-workbench2|5a0db55c7b4751b18f8d2a1ffe3ab3b5813b20c6
Tests developer-tests-workbench2: #663

• Merged main and added runtime status

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.

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)

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!