Project

General

Profile

Actions
Copy link

Bug #5990

closed

[SDKs] arv-run defaults to using arvados/jobs, without checking that it exists or is recent

Added by Brett Smith almost 9 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Peter Amstutz
Category:
SDKs
Target version:
2015-08-05 sprint
Story points:
0.5

Description

This can lead to pipeline failures that are mysterious to users, if arvados/jobs does not exist or has an SDK that's too old to run a current run-command.

Once #3454 is done and deployed, we may be able to rely on that, and take the default Docker image out of arv-run completely.

Subtasks 3 (0 open — 3 closed)

Task #6105: Review 5990-arv-run-default-runtimeResolvedPeter Amstutz05/25/201505/25/2015Actions
Task #6722: FixResolvedPeter Amstutz05/25/2015Actions
Task #6734: Update docsResolvedPeter Amstutz05/25/2015Actions

Related issues

Related to Arvados - Feature #3454: [API] Assign a system-default default docker image (instead of using the non-docker code path) if none is specified in runtime_constraints.ResolvedRadhika Chippada05/07/2015Actions
Follows (1 day) Arvados - Idea #6096: [OPS] Implement a process to regularly deploy a Docker image for running jobs to Arvados clustersResolvedWard Vandewege05/22/2015Actions
Actions
Copy link
 #1

Updated by Brett Smith almost 9 years ago

  • Target version changed from Bug Triage to Arvados Future Sprints
Actions
Copy link
 #2

Updated by Brett Smith almost 9 years ago

  • Target version changed from Arvados Future Sprints to 2015-06-10 sprint
Actions
Copy link
 #3

Updated by Brett Smith almost 9 years ago

  • Story points set to 0.5
Actions
Copy link
 #4

Updated by Tom Clegg almost 9 years ago

Perhaps the default image should have no Arvados SDK, and arv-run should use the version given in the API discovery document as arvados_sdk_version?

Actions
Copy link
 #5

Updated by Peter Amstutz almost 9 years ago

  • Assigned To set to Peter Amstutz
Actions
Copy link
 #6

Updated by Brett Smith almost 9 years ago

  • Assigned To deleted (Peter Amstutz)
  • Target version changed from 2015-06-10 sprint to 2015-07-08 sprint
Actions
Copy link
 #7

Updated by Brett Smith almost 9 years ago

  • Assigned To set to Brett Smith
Actions
Copy link
 #8

Updated by Brett Smith almost 9 years ago

  • Assigned To deleted (Brett Smith)
  • Target version changed from 2015-07-08 sprint to 2015-08-05 sprint
Actions
Copy link
 #9

Updated by Peter Amstutz over 8 years ago

  • Assigned To set to Peter Amstutz
Actions
Copy link
 #10

Updated by Nico César over 8 years ago

I see no unittest for this. Is because they are too hard to reproduce? I did a bunch of greps and I coulnd't find ant "arv run" excecutions on our testsuite

I see you updated doc/user/topics/arv-run.html.textile.liquid

but there is also

doc/sdk/cli/subcommands.html.textile.liquid
arvados/docker/build/doc/sdk/cli/subcommands.html.textile.liquid

that need change.

Actions
Copy link
 #11

Updated by Peter Amstutz over 8 years ago

  • Status changed from New to In Progress
Actions
Copy link
 #12

Updated by Peter Amstutz over 8 years ago

Nico Cesar wrote:

I see no unittest for this. Is because they are too hard to reproduce? I did a bunch of greps and I coulnd't find ant "arv run" excecutions on our testsuite

Unfortunately that's correct, there isn't any automated testing of job running in the test suite, which it pretty hard to have useful tests for tools like arv-run that submit jobs.

I see you updated doc/user/topics/arv-run.html.textile.liquid

but there is also

doc/sdk/cli/subcommands.html.textile.liquid

Fixed.

arvados/docker/build/doc/sdk/cli/subcommands.html.textile.liquid

That's copied from the above file.

Actions
Copy link
 #13

Updated by Nico César over 8 years ago

LGTM

Actions
Copy link
 #14

Updated by Peter Amstutz over 8 years ago

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

Applied in changeset arvados|commit:e69333d856baabf8bbbc27602850dcfe86309d23.

Actions
Copy link

Also available in: Atom PDF