Bug #5990
closed[SDKs] arv-run defaults to using arvados/jobs, without checking that it exists or is recent
100%
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.
Updated by Brett Smith over 9 years ago
- Target version changed from Bug Triage to Arvados Future Sprints
Updated by Brett Smith over 9 years ago
- Target version changed from Arvados Future Sprints to 2015-06-10 sprint
Updated by Tom Clegg over 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?
Updated by Brett Smith over 9 years ago
- Assigned To deleted (
Peter Amstutz) - Target version changed from 2015-06-10 sprint to 2015-07-08 sprint
Updated by Brett Smith over 9 years ago
- Assigned To deleted (
Brett Smith) - Target version changed from 2015-07-08 sprint to 2015-08-05 sprint
Updated by Nico César over 9 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.
Updated by Peter Amstutz over 9 years ago
- Status changed from New to In Progress
Updated by Peter Amstutz over 9 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.
Updated by Peter Amstutz over 9 years ago
- Status changed from In Progress to Resolved
- % Done changed from 67 to 100
Applied in changeset arvados|commit:e69333d856baabf8bbbc27602850dcfe86309d23.