Project

General

Profile

Actions
Copy link

Feature #19093

closed

Process details should display a warning if "container_count" is greater than 1 (meaning it failed and was retried)

Added by Peter Amstutz over 2 years ago. Updated about 2 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Stephen Smith
Category:
Workbench2
Target version:
2022-07-06
Start date:
06/23/2022
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release:
Arvados 2.5.0
Release relationship:
Auto

Subtasks 1 (0 open — 1 closed)

Task #19186: Review 19093-process-failed-warningResolvedLucas Di Pentima06/23/2022

Actions

Related issues 1 (1 open — 0 closed)

Related to Arvados - Story #18860: Display count of container attempts and make it easy to access past logsNew

Actions
Actions
Copy link
 #1

Updated by Peter Amstutz over 2 years ago

  • Target version changed from 2022-05-25 sprint to 2022-06-08 sprint
Actions
Copy link
 #2

Updated by Peter Amstutz over 2 years ago

  • Target version changed from 2022-06-08 sprint to 2022-06-22 Sprint
Actions
Copy link
 #3

Updated by Peter Amstutz over 2 years ago

  • Assigned To set to Stephen Smith
Actions
Copy link
 #4

Updated by Stephen Smith over 2 years ago

  • Status changed from New to In Progress
Actions
Copy link
 #5

Updated by Peter Amstutz over 2 years ago

  • Target version changed from 2022-06-22 Sprint to 2022-07-06
Actions
Copy link
 #6

Updated by Peter Amstutz over 2 years ago

  • Target version changed from 2022-07-06 to 2022-06-22 Sprint
Actions
Copy link
 #7

Updated by Peter Amstutz over 2 years ago

  • Target version changed from 2022-06-22 Sprint to 2022-07-06
Actions
Copy link
 #8

Updated by Stephen Smith over 2 years ago

Changes at arvados-workbench2|3ebe70ccec47fe17491415736701e6e1ce1956c3 branch 19093-process-failed-warning
Tests developer-tests-workbench2: #763

  • Adds an additional process details attributes warning banner when container count > 1
Actions
Copy link
 #9

Updated by Lucas Di Pentima over 2 years ago

Some comments:

  • The data-cy=process-runtime-status-warning property is meant for Cypress tests to find the original runtime status warning, so I think in the case of the container_count warning, it should have its own identifier.
  • The expansion component allows to show additional information that in this case isn't available, do you think it would be better to use for example the Paper component ? For example, I think Paper won't need special CSS rules to disable unwanted pointer behavior.
  • Could you add a Cypress test on process.spec.js?
Actions
Copy link
 #10

Updated by Stephen Smith over 2 years ago

Changes at arvados-workbench2|25d64433dff3ffcf083c2cedd8d849de49a691d1
Tests developer-tests-workbench2: #766

  • Swapped out for Paper
  • Added cypress tests with cy.intercept
Actions
Copy link
 #11

Updated by Lucas Di Pentima over 2 years ago

Really cool what cy.intercept allows us to do! LGTM, thanks.

Actions
Copy link
 #12

Updated by Stephen Smith over 2 years ago

  • Status changed from In Progress to Resolved
Actions
Copy link
 #13

Updated by Stephen Smith about 2 years ago

  • Related to Story #18860: Display count of container attempts and make it easy to access past logs added
Actions
Copy link
 #14

Updated by Peter Amstutz about 2 years ago

  • Release set to 47
Actions
Copy link

Also available in: Atom PDF