Bug #13168
[API] state/priority-change triggers should not change container request modified_by_user_uuid to root
Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
03/29/2018
Due date:
% Done:
100%
Estimated time:
(Total: 0.00 h)
Story points:
-
Release:
Release relationship:
Auto
Description
For example, when a container finishes, each container request with a matching container_uuid gets finalized (log, output, and state are updated). Currently, the finalizing mechanism inadvertently updates the container request's "modified by user" attribute to root. It should be left alone.
Subtasks
Related issues
Associated revisions
History
#1
Updated by Tom Clegg almost 3 years ago
- Description updated (diff)
#2
Updated by Tom Clegg almost 3 years ago
- Related to Story #10511: [Crunch2] [API] Specify which user's credentials should be used by a container added
#3
Updated by Tom Morris almost 3 years ago
- Target version set to 2018-04-11 Sprint
#4
Updated by Tom Clegg almost 3 years ago
- Has duplicate Feature #13279: [Crunch v2] add job field in crunch v2 to show who created job added
#5
Updated by Lucas Di Pentima almost 3 years ago
- Assigned To set to Lucas Di Pentima
#6
Updated by Lucas Di Pentima almost 3 years ago
- Status changed from New to In Progress
#7
Updated by Lucas Di Pentima almost 3 years ago
Updates at 378eeb13b - branch 13168-cr-modified-user
Test run: https://ci.curoverse.com/job/developer-run-tests/672/
#8
Updated by Peter Amstutz almost 3 years ago
I did a test running a container, the container request modified_at had the right user after it finished, so this looks LGTM.
#9
Updated by Lucas Di Pentima almost 3 years ago
- Status changed from In Progress to Resolved
Applied in changeset arvados|2baac8a6702551064b065f752f75a9b40dbec0f5.
#10
Updated by Tom Morris over 2 years ago
- Release set to 13
#11
Updated by Tom Clegg about 2 years ago
- Related to Bug #14595: [API] logs.event_at has wrong timestamp added
Merge branch '13168-cr-modified-user'
Closes #13168
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima@veritasgenetics.com>