Actions
Bug #10216
closed
[Crunch2] [API] Allow container request to bypass reuse (ensure a new container is created)
Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
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:
- Add a 'use_existing' flag, which defaults to true. When set false a new container should always be created.
- Add '"use_existing": false, ' to the 'Hello, Crunch' example at http://doc.arvados.org/install/crunch2-slurm/install-test.html
As for the future "filters" parameter, if/when that gets implemented, the 'use_existing' flag should take precedence if set to false.
Related issues
Updated by
Updated by Tom Morris over 7 years ago
- Description updated (diff)
- Target version set to 2016-10-26 sprint
- Story points set to 0.5
Updated by Lucas Di Pentima over 7 years ago
- Assigned To set to Lucas Di Pentima
Updated by Lucas Di Pentima over 7 years ago
- Status changed from New to In Progress
Updated by Lucas Di Pentima over 7 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/
Updated by Radhika Chippada over 7 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.
Updated by Lucas Di Pentima over 7 years ago
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Applied in changeset arvados|commit:e9b2c5fc743957598d839e072c49e027752b3287.
Actions