Project

General

Profile

Actions

Bug #19315

closed

Display "runtime_user_uuid" in details

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

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Target version:
Start date:
10/27/2022
Due date:
% Done:

100%

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

Description

From discussion:

What we actually want to show is modified_by_user_uuid (but call it something like "user who submitted the container". We only want to show "runtime_user_uuid" (meaning "user who the container is actually running as") if it is different.

modified_by_user_uuid -> "Submitted by"

runtime_user_uuid -> "Run as"

Only show "Run as" if it is different than "Submitted by"


Subtasks 1 (0 open1 closed)

Task #19318: Review 19315-process-runtime-user-rebase1ResolvedPeter Amstutz10/27/2022

Actions
Actions #1

Updated by Peter Amstutz over 2 years ago

  • Description updated (diff)
Actions #2

Updated by Peter Amstutz over 2 years ago

  • Assigned To set to Peter Amstutz
Actions #3

Updated by Peter Amstutz over 2 years ago

  • Assigned To changed from Peter Amstutz to Stephen Smith
Actions #4

Updated by Peter Amstutz over 2 years ago

  • Target version changed from 2022-08-17 sprint to 2022-08-31 sprint
Actions #5

Updated by Peter Amstutz over 2 years ago

  • Target version changed from 2022-08-31 sprint to 2022-09-14 sprint
Actions #6

Updated by Stephen Smith over 2 years ago

  • Status changed from New to In Progress
Actions #7

Updated by Stephen Smith over 2 years ago

Changes at arvados-workbench2|dab0ef26fc6ba959d60a31ce64f5a2931580d30f branch 19315-process-runtime-user
Tests developer-tests-workbench2: #874

  • Displays runtime user full name with uuid in details attributes
  • Add cypress test to check details attributes
Actions #8

Updated by Peter Amstutz over 2 years ago

  • Target version changed from 2022-09-14 sprint to 2022-10-12 sprint
Actions #9

Updated by Peter Amstutz over 2 years ago

  • Description updated (diff)
Actions #10

Updated by Peter Amstutz about 2 years ago

  • Target version changed from 2022-10-12 sprint to 2022-10-26 sprint
Actions #11

Updated by Peter Amstutz about 2 years ago

  • Target version changed from 2022-10-26 sprint to 2022-11-09 sprint
Actions #12

Updated by Peter Amstutz about 2 years ago

  • Description updated (diff)
Actions #13

Updated by Stephen Smith about 2 years ago

Changes at 7abad6fba6ffa8e88d40ace78a84fb2a99d5d44b branch 19315-process-runtime-user-rebase1
Tests developer-tests-workbench2: #984

  • Added modified by (labeled Submitted by)
  • Hide runtime user (Run as) when equal to modified by
Actions #14

Updated by Peter Amstutz about 2 years ago

Stephen Smith wrote in #note-13:

Changes at 7abad6fba6ffa8e88d40ace78a84fb2a99d5d44b branch 19315-process-runtime-user-rebase1
Tests developer-tests-workbench2: #984

  • Added modified by (labeled Submitted by)
  • Hide runtime user (Run as) when equal to modified by

You need to use modified_by_user_uuid from the container request, not the container. That's because for containers, the modified_by is always the system user.

Actions #15

Updated by Stephen Smith about 2 years ago

Changes at arvados-workbench2|f15f6c31dc4eb0c606e448fdee1391bfb877c007
Tests developer-tests-workbench2: #992

  • Show modifiedby from containerrequest instead
  • Test for both modifiedby same and different from runtimeuser
  • Add a couple extra seconds of timeout for flaky process log tests
Actions #16

Updated by Peter Amstutz about 2 years ago

Stephen Smith wrote in #note-15:

Changes at arvados-workbench2|f15f6c31dc4eb0c606e448fdee1391bfb877c007
Tests developer-tests-workbench2: #992

  • Show modifiedby from containerrequest instead
  • Test for both modifiedby same and different from runtimeuser
  • Add a couple extra seconds of timeout for flaky process log tests

This LGTM!

Actions #17

Updated by Stephen Smith about 2 years ago

  • Status changed from In Progress to Resolved
Actions #18

Updated by Peter Amstutz about 2 years ago

  • Release set to 47
Actions

Also available in: Atom PDF