Project

General

Profile

Actions

Feature #3454

closed

[API] Assign a system-default default docker image (instead of using the non-docker code path) if none is specified in runtime_constraints.

Added by Peter Amstutz over 9 years ago. Updated almost 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Radhika Chippada
Category:
API
Target version:
Story points:
0.5

Subtasks 1 (0 open1 closed)

Task #5942: Review branch: 3454-default-docker-imageResolvedRadhika Chippada05/07/2015Actions

Related issues

Related to Arvados - Bug #5990: [SDKs] arv-run defaults to using arvados/jobs, without checking that it exists or is recentResolvedPeter Amstutz05/25/201505/25/2015Actions
Related to Arvados - Feature #6348: [Deployment] [Documentation] Minimize system-wide dependencies for compute node setupNewActions
Actions #1

Updated by Peter Amstutz over 9 years ago

  • Subject changed from Reject jobs that are missing docker_image in runtime_constraints. to [API] Reject jobs that are missing docker_image in runtime_constraints.
  • Category set to API
Actions #2

Updated by Tom Clegg over 9 years ago

  • Tracker changed from Bug to Feature
Actions #3

Updated by Tom Clegg over 9 years ago

  • Target version set to Arvados Future Sprints
Actions #4

Updated by Tom Clegg almost 9 years ago

  • Subject changed from [API] Reject jobs that are missing docker_image in runtime_constraints. to [API] Assign a system-default default docker image (instead of using the non-docker code path) if none is specified in runtime_constraints.
  • Story points set to 0.5
Actions #5

Updated by Tom Clegg almost 9 years ago

  • Target version changed from Arvados Future Sprints to 2015-05-20 sprint
Actions #6

Updated by Radhika Chippada almost 9 years ago

  • Assigned To set to Radhika Chippada
Actions #7

Updated by Radhika Chippada almost 9 years ago

  • Status changed from New to In Progress
Actions #8

Updated by Peter Amstutz almost 9 years ago

  • If Rails.configuration.default_docker_image_for_jobs is false, and docker_image is not set in the job, that should probably be a validation error instead of falling back to the old behavior (running in a non-Docker configuration)
  • Instead of runtime_constraints.is_a? Hash suggest setting runtime_constraints ||= {}
Actions #9

Updated by Radhika Chippada almost 9 years ago

I observed that making the job invalid when "no default config + runtime_constraints not including docker image" is a very drastic change. A large number of tests failed without the default config.

This made me wonder if we will have the very undesirable side effect of not being able to access jobs when the system does not have the default config.

I discussed this with Peter, and we decided to not do this as part of this update (the feature does not ask for it).

If we want this, we will address it in a different feature request.

Actions #10

Updated by Radhika Chippada almost 9 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:1cea19230b490876f99194ea4d84970d6330ae12.

Actions

Also available in: Atom PDF