Project

General

Profile

Actions

Bug #20251

closed

Subprocess list show 'Unknown' status labels on some items, gets corrected when clicking through

Added by Lucas Di Pentima about 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Story points:
-
Release relationship:
Auto

Description

When visiting a running workflow's process page that has many thousands subprocesses and looking at the Subprocesses Panel with page size >= 200, some subprocesses show as having the Status=Unknown. When clicking through, the correct status is displayed and going back to its parent process make the listing show its correct status.

For some reason, I wasn't able to get this behavior when using page sizes of less than 200, so it may be related to the size of the filtering parameters that get used to retrieve the containers/CRs from the middleware.

The subprocess list is ordered by created_at by default, if I set the page size to 200 and scroll through the first page's content, I get an 'Unknown' label at the 101th element, but when using page size 100, or 50 and then go to that 101th element, it shows its correct status without needing to click through it.


Subtasks 1 (0 open1 closed)

Task #20261: Review 20251-subprocess-panel-container-details-bugResolvedStephen Smith03/22/2023Actions
Actions #1

Updated by Stephen Smith about 1 year ago

  • Assigned To set to Stephen Smith
Actions #2

Updated by Stephen Smith about 1 year ago

  • Target version changed from Future to Development 2023-03-29 Sprint
Actions #3

Updated by Peter Amstutz about 1 year ago

  • Release set to 57
Actions #4

Updated by Stephen Smith about 1 year ago

Changes at arvados-workbench2|d3518cf6eb602101cad0fb90c534499b7eb6cbca branch 20251-subprocess-panel-container-details-bug
Tests developer-tests-workbench2: #1167

  • Set container list query limit to match the requested item list, which reflects the page size of the data explorer making the request
  • Remove duplicate UUIDs from request
  • Improve flaky clipboard test
  • Fix race condition in collection file browser by using the callback form of setState when referencing previous state
Actions #5

Updated by Stephen Smith about 1 year ago

  • Status changed from New to In Progress
Actions #6

Updated by Lucas Di Pentima about 1 year ago

Just one small suggestion:

  • I think you can improve the deduplication performance by instead of using includes() on every element, passing this to loadContainers(): [...new Set(containerUuids)], wdyt?

The rest LGTM, thanks!

Actions #7

Updated by Stephen Smith about 1 year ago

  • Status changed from In Progress to Resolved
  • Keeping current implementation since a benchmark shows consistent better performance in firefox using .includes, no significant difference in chrome

Applied in arvados-workbench2|f0cfaf399e8476e534781dbdc69ec6dc1133308b

Actions

Also available in: Atom PDF