Bug #13168

[API] state/priority-change triggers should not change container request modified_by_user_uuid to root

Added by Tom Clegg almost 3 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Lucas Di Pentima
Category:
-
Target version:
2018-04-11 Sprint
Start date:
03/29/2018
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release:
Arvados 1.2
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

Task #13283: Review 13168-cr-modified-userResolvedLucas Di Pentima

Related issues

Related to Arvados - Story #10511: [Crunch2] [API] Specify which user's credentials should be used by a containerResolved

Related to Arvados - Bug #14595: [API] logs.event_at has wrong timestampResolved12/13/2018

Has duplicate Arvados - Feature #13279: [Crunch v2] add job field in crunch v2 to show who created jobDuplicate

Associated revisions

Revision 2baac8a6
Added by Lucas Di Pentima almost 3 years ago

Merge branch '13168-cr-modified-user'
Closes #13168

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <>

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

#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

Also available in: Atom PDF