Peter Amstutz [Mon, 21 Mar 2016 20:24:06 +0000 (16:24 -0400)]
8654: Scanning for and uploading dependencies for submit works. Add test for
--submit. Fix cwl-runner to handle CWL File objects in script_parameters.
Peter Amstutz [Mon, 21 Mar 2016 16:37:38 +0000 (12:37 -0400)]
Merge branch 'master' into 8654-arv-jobs-cwl-runner
Ward Vandewege [Sun, 20 Mar 2016 18:39:52 +0000 (14:39 -0400)]
More small build script improvements.
No issue #
Ward Vandewege [Sun, 20 Mar 2016 17:16:18 +0000 (13:16 -0400)]
Build script improvements:
* move some more functions into run-library.sh
* add a wrapper script for our single target package build/test/upload step
No issue #
Peter Amstutz [Sat, 19 Mar 2016 20:00:11 +0000 (16:00 -0400)]
Enable BaseComputeNodeDriver.list_nodes to pass through keyword arguments.
closes #8761
Brett Smith [Fri, 18 Mar 2016 18:50:59 +0000 (14:50 -0400)]
Merge branch '8002-no-pyvcf-wip'
Closes #8002, #8759.
Brett Smith [Fri, 18 Mar 2016 18:37:13 +0000 (14:37 -0400)]
8002: Stop backporting PyVCF.
Arvados proper doesn't use it. We only backported it as a convenience
previously.
Brett Smith [Fri, 18 Mar 2016 15:31:00 +0000 (11:31 -0400)]
Merge branch '7852-readline-cache-fix-wip'
Closes #7852, #8755.
Brett Smith [Thu, 17 Mar 2016 22:03:21 +0000 (18:03 -0400)]
7852: Bugfix _filepos tracking in PySDK readline method.
Make sure self._filepos is always in the right position to return
uncached data before another read method could be called, to avoid
inconsistent results when interleaving those calls with readline
calls.
radhika [Wed, 3 Feb 2016 17:19:54 +0000 (12:19 -0500)]
7852: Added one more test "test_readline_then_readall" which does a readline followed by a readall.
This test also fails exactly like the test "test_readline_then_readlines".
radhika [Mon, 25 Jan 2016 21:37:24 +0000 (16:37 -0500)]
7852: add a failing test that does: readline and then readlines. The expectation would be the second call,
readlines, would fetch all the remaining lines after the first line. However, the readlines call is only
fetching the data after the "first block" which was read during the first readline call. It appears that
self._readline_cache and self._filepos are playing a role here, which are set during the first readline.
Peter Amstutz [Fri, 18 Mar 2016 14:21:55 +0000 (10:21 -0400)]
Fix log scanning for "crunchrunner: $(task.outdir)" performed by arvados cwl
runner. no issue #
Ward Vandewege [Fri, 18 Mar 2016 02:19:24 +0000 (22:19 -0400)]
Make the build of the schema_salad packages more quiet.
refs #8671
Tom Clegg [Thu, 17 Mar 2016 22:15:10 +0000 (18:15 -0400)]
gofmt. refs #8754
Peter Amstutz [Thu, 17 Mar 2016 21:22:30 +0000 (17:22 -0400)]
8654: Resource upload WIP
Peter Amstutz [Thu, 17 Mar 2016 20:34:48 +0000 (16:34 -0400)]
Improve exception reporting for arvados-cwl-runner, no issue #
Ward Vandewege [Thu, 17 Mar 2016 19:45:54 +0000 (15:45 -0400)]
Build dependencies for arvados-cwl-runner, automatically.
closes #8671
Brett Smith [Thu, 17 Mar 2016 18:42:00 +0000 (14:42 -0400)]
8752: Skip unreliable Python SDK Keep test.
Refs #8752. See there for more discussion about the circumstances of
failure.
radhika [Thu, 17 Mar 2016 15:30:01 +0000 (11:30 -0400)]
closes #8561
Merge branch '8561-node-pairing'
radhika [Thu, 17 Mar 2016 15:29:44 +0000 (11:29 -0400)]
Merge branch 'master' into 8561-node-pairing
radhika [Thu, 17 Mar 2016 15:28:23 +0000 (11:28 -0400)]
8561: delete redundant "maxDiff = None"
radhika [Thu, 17 Mar 2016 15:17:12 +0000 (11:17 -0400)]
Merge branch '8561-node-pairing' of git.curoverse.com:arvados into 8561-node-pairing
Peter Amstutz [Thu, 17 Mar 2016 14:42:02 +0000 (10:42 -0400)]
8561: Set ec2_instance_id and cloud_factory().node_id to mock cloud/arvados id
instead of constant '2'.
Peter Amstutz [Thu, 17 Mar 2016 13:59:10 +0000 (09:59 -0400)]
Node manager bugfix: late subscribers should get proxy() not _later (which is a tell_proxy())
fixes #8691 note-4
Brett Smith [Thu, 17 Mar 2016 13:57:26 +0000 (09:57 -0400)]
Merge branch '8709-docker-cleaner-volumes-wip'
Closes #8709, #8711.
Brett Smith [Tue, 15 Mar 2016 21:59:16 +0000 (17:59 -0400)]
8709: Docker cleaner cleans container volumes.
This prevents a compute node's Docker partition from filling with
scratch space volumes.
radhika [Wed, 16 Mar 2016 19:46:38 +0000 (15:46 -0400)]
Merge branch 'master' into 8561-node-pairing
radhika [Wed, 16 Mar 2016 19:22:14 +0000 (15:22 -0400)]
closes #8554
Merge branch '8554-trash-untrash-unix-volume'
radhika [Wed, 16 Mar 2016 19:21:10 +0000 (15:21 -0400)]
Merge branch 'master' into 8554-trash-untrash-unix-volume
Tom Clegg [Wed, 16 Mar 2016 16:54:55 +0000 (12:54 -0400)]
8554: Report Walk errors and continue, instead of stopping silently.
Tom Clegg [Wed, 16 Mar 2016 16:41:56 +0000 (12:41 -0400)]
8554: Un-pyramid code
Tom Clegg [Wed, 16 Mar 2016 16:16:15 +0000 (12:16 -0400)]
8554: Dedup Get() checks, add comments, fix up regexp.
Peter Amstutz [Wed, 16 Mar 2016 16:11:34 +0000 (12:11 -0400)]
8561: Fix test_arvados_node_match
Tom Clegg [Wed, 16 Mar 2016 07:43:35 +0000 (03:43 -0400)]
Fix backports dir search for PACKAGE="llfuse==0.41.1".
Also, drop "-v 0.41.1" args from fpm-info.sh: cannot specify version
both here and in run-build-packages.sh.
refs #8345
Peter Amstutz [Wed, 16 Mar 2016 01:48:32 +0000 (21:48 -0400)]
Merge branch '6518-crunch2-dispatch-slurm' closes #6518
Peter Amstutz [Wed, 16 Mar 2016 01:43:52 +0000 (21:43 -0400)]
6518: Use ARVADOS_API_HOST when checking strigger command line in test.
Peter Amstutz [Wed, 16 Mar 2016 01:29:28 +0000 (21:29 -0400)]
6518: Add check for sbatch/strigger command line in test.
Peter Amstutz [Wed, 16 Mar 2016 00:03:40 +0000 (20:03 -0400)]
6518: Add crunch-dispatch-slurm to tests.
Peter Amstutz [Wed, 16 Mar 2016 00:02:30 +0000 (20:02 -0400)]
Merge branch 'master' into 6518-crunch2-dispatch-slurm
Peter Amstutz [Wed, 16 Mar 2016 00:01:19 +0000 (20:01 -0400)]
6518: Merge in tests. Code cleanup around variable naming and comments.
radhika [Tue, 15 Mar 2016 20:23:58 +0000 (16:23 -0400)]
8561: test tweaks
Tom Clegg [Tue, 15 Mar 2016 20:09:20 +0000 (16:09 -0400)]
Merge branch '8491-import-build-tools' refs #8491
Tom Clegg [Tue, 15 Mar 2016 20:09:03 +0000 (16:09 -0400)]
8491: Add default target "make help"
radhika [Tue, 15 Mar 2016 15:27:08 +0000 (11:27 -0400)]
8554: removed testTrashUntrashWithEmptyTrashGoroutine
Peter Amstutz [Tue, 15 Mar 2016 13:34:36 +0000 (09:34 -0400)]
Proxy objects held in node manager dict of booting nodes should be regular proxy(), not
tell_proxy(). fixes #8691
Brett Smith [Tue, 15 Mar 2016 01:05:17 +0000 (21:05 -0400)]
8203/8690: crunch-job uses `srun --nodes=1`.
Using `srun --nodelist=[one node]` is not sufficient to ensure that
the command runs on a single node; only that it will at least be
allocated the requested node. This is documented in the srun man
page, and we've observed additional node allocations in these calls.
When srun allocates additional nodes to the work, this can break later
parsing code in crunch-job.
Prefer --nodes, which sets a hard maximum on the number of nodes used.
We're already using this to run individual tasks (-N1), so it's a
known strategy. Tested live before pushing. Refs #8203, #8690.
radhika [Tue, 15 Mar 2016 00:19:28 +0000 (20:19 -0400)]
8561: rather than getting node's ipadd, use it's ec2_instance_id
Tom Clegg [Mon, 14 Mar 2016 19:54:32 +0000 (15:54 -0400)]
Fix ineffective test case
No issue #
Tom Clegg [Mon, 14 Mar 2016 19:06:02 +0000 (15:06 -0400)]
Merge branch '8079-lookup-token-uuid' closes #8079
Tom Clegg [Mon, 14 Mar 2016 19:04:12 +0000 (15:04 -0400)]
8079: Allow where(api_token: foo) and disallow where(api_token: ["contains", "f"])
Tom Clegg [Sat, 12 Mar 2016 22:32:17 +0000 (17:32 -0500)]
8079: Tidy up and document current_api_client_is_trusted.
Tom Clegg [Sun, 6 Mar 2016 19:50:22 +0000 (14:50 -0500)]
8079: Prevent users from changing their own token UUIDs.
Tom Clegg [Fri, 4 Mar 2016 22:30:53 +0000 (17:30 -0500)]
8079: Prevent users from looking up other users' tokens by UUID.
Previous code was allowing any user logging in through a "trusted
client" (typically Workbench) to retrieve the secret token for any
ApiClientAuthorization whose UUID is known. This won't be acceptable
when Container records start including those UUIDs.
Also added permission for any user to update (e.g., change expiration)
and delete their current token, even if the token wasn't assigned
through a "trusted client".
Peter Amstutz [Mon, 14 Mar 2016 17:19:56 +0000 (13:19 -0400)]
Really fix cwl-runner tests this time. refs #8673
Peter Amstutz [Mon, 14 Mar 2016 16:21:31 +0000 (12:21 -0400)]
Fix cwl-runner tests. refs #8673
Peter Amstutz [Mon, 14 Mar 2016 14:20:11 +0000 (10:20 -0400)]
Merge branch '8673-cwl-runner-project-uuid' closes #8673
radhika [Mon, 14 Mar 2016 14:02:50 +0000 (10:02 -0400)]
8554: test improvements
radhika [Mon, 14 Mar 2016 04:22:15 +0000 (00:22 -0400)]
8554: improved tests
radhika [Sun, 13 Mar 2016 12:52:49 +0000 (08:52 -0400)]
8554: Untrash returns os.ErrNotExist when no trash found for the given locator and other updates.
Peter Amstutz [Fri, 11 Mar 2016 22:35:09 +0000 (17:35 -0500)]
8654: Collect output files into output collection, use as output for task.
Peter Amstutz [Fri, 11 Mar 2016 21:55:23 +0000 (16:55 -0500)]
Merge branch '8673-cwl-runner-project-uuid' into 8654-arv-jobs-cwl-runner
Peter Amstutz [Fri, 11 Mar 2016 21:54:59 +0000 (16:54 -0500)]
8654: Passes in local testing.
Peter Amstutz [Fri, 11 Mar 2016 20:33:30 +0000 (15:33 -0500)]
Fix proxy/tell_proxy usage in _begin_node_shutdown, refs #8678
Peter Amstutz [Fri, 11 Mar 2016 19:37:13 +0000 (14:37 -0500)]
8673: Fix --project-uuid for arv-keepdocker upload.
Tom Clegg [Fri, 11 Mar 2016 19:16:48 +0000 (14:16 -0500)]
8491: Merge branch 'master' into 8491-import-build-tools
Tom Clegg [Wed, 9 Mar 2016 21:01:22 +0000 (16:01 -0500)]
8491: 8222: Prohibit activesupport 4.2.6 so our gem can be installed with ruby <2.2.2
Tom Clegg [Wed, 9 Mar 2016 18:36:49 +0000 (13:36 -0500)]
8491: Remove git-commit.version before running api server tests
Tom Clegg [Wed, 9 Mar 2016 18:36:16 +0000 (13:36 -0500)]
8491: --retry applies to do_install too
Tom Clegg [Tue, 8 Mar 2016 21:37:42 +0000 (16:37 -0500)]
8491: move files from arvados-dev into their new places
Tom Clegg [Fri, 11 Mar 2016 18:40:36 +0000 (13:40 -0500)]
Tom Clegg [Fri, 11 Mar 2016 18:35:13 +0000 (13:35 -0500)]
Merge branch '8345-llfuse-unpin' refs #8345
Tom Clegg [Fri, 11 Mar 2016 17:28:53 +0000 (12:28 -0500)]
8345: Need "yum install -y" to reliably install rpm packages.
Tom Clegg [Fri, 11 Mar 2016 07:15:10 +0000 (02:15 -0500)]
8345: 8491: Downgrade to llfuse 0.41 until #8345 gets sorted out (but keep the fuse backports).
Peter Amstutz [Fri, 11 Mar 2016 18:31:50 +0000 (13:31 -0500)]
8673: Report pipeline instance uuid. Reuse jobs by default. Bump cwltool
version dependency.
radhika [Fri, 11 Mar 2016 14:46:47 +0000 (09:46 -0500)]
closes #8558
Merge branch '8558-cwl-propagate-resource-req'
radhika [Fri, 11 Mar 2016 14:45:17 +0000 (09:45 -0500)]
Merge branch 'master' into 8558-cwl-propagate-resource-req
Conflicts:
sdk/cwl/arvados_cwl/__init__.py
Peter Amstutz [Fri, 11 Mar 2016 13:00:52 +0000 (08:00 -0500)]
8673: Typo fix for --project-uuid. Refactor main loop locking to address
possible race condition causing workflow to end prematurely.
radhika [Fri, 11 Mar 2016 03:24:06 +0000 (22:24 -0500)]
6518: create sbatch and strigger commands using func declarations so that they can be overridden in tests.
Peter Amstutz [Thu, 10 Mar 2016 20:54:27 +0000 (15:54 -0500)]
Fix node manager to send ActorProxy instead of TellActorProxy to
ComputeNodeStateChangeBase subscribers. refs #8543
Ward Vandewege [Thu, 10 Mar 2016 19:06:38 +0000 (14:06 -0500)]
Get the go binary from a http url instead of https, so that our
transparent proxy can cache it.
No issue #
Ward Vandewege [Thu, 10 Mar 2016 19:05:21 +0000 (14:05 -0500)]
Build package for arvados-cwl-runner.
refs #8671
radhika [Thu, 10 Mar 2016 17:44:22 +0000 (12:44 -0500)]
8558: when cores not found in resources, use default.
Peter Amstutz [Thu, 10 Mar 2016 16:57:57 +0000 (11:57 -0500)]
8673: Adding --project-uuid so resources are created in the specified project.
radhika [Thu, 10 Mar 2016 16:39:40 +0000 (11:39 -0500)]
refs #8558
Merge branch '8558-add-cwl-jenkins'
radhika [Thu, 10 Mar 2016 16:36:12 +0000 (11:36 -0500)]
8558: compacted code
Peter Amstutz [Thu, 10 Mar 2016 14:42:40 +0000 (09:42 -0500)]
Add Dockerfile & update sources.list
Peter Amstutz [Thu, 10 Mar 2016 13:28:58 +0000 (08:28 -0500)]
Fixes
Peter Amstutz [Thu, 10 Mar 2016 13:03:39 +0000 (08:03 -0500)]
Stuff
Peter Amstutz [Wed, 9 Mar 2016 22:44:36 +0000 (17:44 -0500)]
8654: cwl-runner crunch script
Peter Amstutz [Wed, 9 Mar 2016 22:02:05 +0000 (17:02 -0500)]
8654: cwl-runner crunch stub.
Peter Amstutz [Wed, 9 Mar 2016 20:32:52 +0000 (15:32 -0500)]
8636: Fix ArvPathMapper.reversemap to return a tuple as expected by caller.
Improve job reuse by setting minimum_script_version. refs #8636
radhika [Wed, 9 Mar 2016 19:39:58 +0000 (14:39 -0500)]
8558: update field names and ensure the tests pass
radhika [Wed, 9 Mar 2016 19:39:30 +0000 (14:39 -0500)]
8558: add cwl to jenkins
Tom Clegg [Wed, 9 Mar 2016 18:58:29 +0000 (13:58 -0500)]
Remove duplicate environment variable.
Fixes tests failing after perl security fix.
"Previously on a duplicate, %ENV would have the last entry for the name
from environ[], but a typical getenv() would return the first entry."
refs:
* http://perl5.git.perl.org/perl.git/commitdiff/
ae37b791a73a9e78dedb89fb2429d2628cf58076
* https://security-tracker.debian.org/tracker/CVE-2016-2381
No issue #
radhika [Wed, 9 Mar 2016 16:43:18 +0000 (11:43 -0500)]
closes #8508
Merge branch 'wtsi-hgi-8508-datamanager-test-badpaths'
radhika [Wed, 9 Mar 2016 16:41:57 +0000 (11:41 -0500)]
8508: go fmt
radhika [Wed, 9 Mar 2016 16:29:57 +0000 (11:29 -0500)]
8554: Add EmptyTrash api to Volume and implementation in volume_unix. Add emptyTrash goroutine.
Peter Amstutz [Tue, 8 Mar 2016 20:08:14 +0000 (15:08 -0500)]
Merge branch '8543-nodemanager-fewer-futures' closes #8543
Peter Amstutz [Thu, 3 Mar 2016 18:54:30 +0000 (13:54 -0500)]
8543: Implement TellActorProxy which uses Actor.tell() instead of Actor.ask().
Implement BaseNodeMangerActor and TellableActorRef to provide tell_proxy().
Convert proxies in Node Manager that don't return Futures to use
TellActorProxy.
Brett Smith [Tue, 8 Mar 2016 15:27:02 +0000 (10:27 -0500)]
Merge branch '8439-gitolite-nonff-perms-wip'
Closes #8439, #8525.