Bug #10216

[Crunch2] [API] Allow container request to bypass reuse (ensure a new container is created)

Added by Tom Clegg almost 5 years ago. Updated almost 5 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
10/10/2016
Due date:
% Done:

100%

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

Description

Example use cases:
  • Container is expected/intended to be impure (results differ from one run to the next)
  • Test/diagnostics container

The Containers API specifies a "filters" parameter which (when implemented) could be used to achieve this with something like filters=[[created_at,=,now]].

Implementation:

As for the future "filters" parameter, if/when that gets implemented, the 'use_existing' flag should take precedence if set to false.


Subtasks

Task #10250: Review 10216-allow-cr-bypass-reuseResolvedRadhika Chippada


Related issues

Related to Arvados - Bug #3555: [API] Rename nondeterministic to impureClosed

Related to Arvados - Feature #8569: [Crunch2] Impure mount from host fsNew

Related to Arvados - Feature #8570: [Crunch2] Impure access to object storeNew

Associated revisions

Revision e9b2c5fc
Added by Lucas Di Pentima almost 5 years ago

Merge branch '10216-allow-cr-bypass-reuse'
Closes #10216

Revision ad825d1c (diff)
Added by Peter Amstutz almost 5 years ago

Permit use_existing in create-and-commit container requests. refs #10216

History

#1 Updated by Ward Vandewege almost 5 years ago

  • Description updated (diff)

#2 Updated by Ward Vandewege almost 5 years ago

  • Description updated (diff)

#3 Updated by Tom Morris almost 5 years ago

  • Description updated (diff)
  • Target version set to 2016-10-26 sprint
  • Story points set to 0.5

#4 Updated by Lucas Di Pentima almost 5 years ago

  • Assigned To set to Lucas Di Pentima

#5 Updated by Lucas Di Pentima almost 5 years ago

  • Status changed from New to In Progress

#6 Updated by Lucas Di Pentima almost 5 years ago

Branch 10216-allow-cr-bypass-reuse at: b1578a5

Added use_existing field to CRs, with default value being "true". When false, the reuse_container mechanism is bypassed.

Tests run at: https://ci.curoverse.com/job/developer-run-tests/39/

#7 Updated by Radhika Chippada almost 5 years ago

Just one minor comment

  • Name of the test “Container request doesn't reuse container when explicitly asked” is a bit confusing. Can we say “when asked not to” or something?

LGTM.

#8 Updated by Lucas Di Pentima almost 5 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:e9b2c5fc743957598d839e072c49e027752b3287.

Also available in: Atom PDF