[ARVADOS] updated: 18f6483ffe8d6191bce55c464070f9f5c08b9b01
Git user
git at public.curoverse.com
Mon Jul 17 13:18:43 EDT 2017
Summary of changes:
AUTHORS | 1 +
build/build.list | 2 +-
build/package-build-dockerfiles/centos7/Dockerfile | 15 +-
build/package-build-dockerfiles/debian8/Dockerfile | 7 +-
.../ubuntu1204/Dockerfile | 7 +-
.../ubuntu1404/Dockerfile | 9 +-
.../ubuntu1604/Dockerfile | 7 +-
build/package-test-dockerfiles/centos7/Dockerfile | 2 +-
build/package-test-dockerfiles/debian8/Dockerfile | 6 +-
.../package-test-dockerfiles/ubuntu1204/Dockerfile | 6 +-
.../package-test-dockerfiles/ubuntu1404/Dockerfile | 6 +-
.../package-test-dockerfiles/ubuntu1604/Dockerfile | 4 +-
build/run-build-packages.sh | 2 +-
sdk/cwl/arvados_cwl/__init__.py | 11 +-
sdk/cwl/arvados_cwl/arvtool.py | 2 +-
sdk/cwl/arvados_cwl/fsaccess.py | 2 +-
sdk/cwl/arvados_cwl/runner.py | 30 +++-
sdk/cwl/setup.py | 4 +-
sdk/cwl/tests/test_job.py | 2 +-
sdk/cwl/tests/test_submit.py | 63 ++++++--
sdk/cwl/tests/wf/expect_packed.cwl | 5 +-
sdk/python/arvados/commands/put.py | 2 +-
sdk/python/tests/test_arv_get.py | 7 +-
sdk/python/tests/test_arv_put.py | 4 +-
services/api/app/models/container_request.rb | 5 +
services/api/test/fixtures/container_requests.yml | 19 +++
services/api/test/fixtures/containers.yml | 17 ++
services/api/test/unit/container_request_test.rb | 24 +++
services/ws/event_source.go | 1 +
services/ws/permission.go | 3 +-
services/ws/session_v0.go | 43 +++--
services/ws/session_v0_test.go | 176 +++++++++++++++------
32 files changed, 382 insertions(+), 112 deletions(-)
via 18f6483ffe8d6191bce55c464070f9f5c08b9b01 (commit)
via 85bd5f3272cfc8ba8e54f4f47ea9865d7de62779 (commit)
via 3a5b003b5cf784e96799d8ddadc939834620fd34 (commit)
via 63e12fa3f15d417526db4f743108fe4eedcfcad0 (commit)
via bf87f4aba81caff15b174393faf074cf9d8ba554 (commit)
via 41f48ea20b33e59ced47a6e96f8198a76d43b619 (commit)
via 6ff6c7a392a7064cba408b566a8ce5b86dadbc78 (commit)
via 96ae27be78eeee85d9462123367b1221d5b96350 (commit)
via eb66b34d22cef6c98ad9ebdc228cf784b3136b72 (commit)
via d3700e50bcaa9db177346199298913b54e37a2d5 (commit)
via 938a0cc22ad9d520c653b8b41d5b1bb2163a1020 (commit)
via b8977c89db2707dd713cd4eeb11d595717fcff75 (commit)
via a95c9948e04eab66cbb3aa4e93776c52956fc807 (commit)
via b6ad461076bdc4830fe08c01d70ed3def1ec6b55 (commit)
via 5d081c423f314060cefafc7149850ea1dcbe098a (commit)
via d1562916e4792c0a9b4b2a4aea842e7c2848d38a (commit)
via 1be8ac0f887cb1c639ac502c47438a8260735069 (commit)
via 1e7fda3fcfb9b3030c65f56e06764d20496a0610 (commit)
via 281cf5782f0e05133abbd7c66db729bcc52b08e4 (commit)
via 48cfc124afff473d58bcef61ec056f18eeaf06c9 (commit)
via bd29cfa5cdba4365faef20003e8856946b7b928c (commit)
via 187382bb88eee65d887c004d73eddf46cbd86bc2 (commit)
via 0041af13b61716fb9044c050b2ef34a3315566d6 (commit)
via a3bfffe950102b82b6655b91038bba9e6b085f03 (commit)
via 2feec8cbac7ef28a47f3c3a9d071b070ef38cb6e (commit)
via 7e38c190147681b7d2ce64601b1ac9661812fcd6 (commit)
via d1a7f9691ff2b9d77106f953c714b1455b228c64 (commit)
via e953b9ecd6815968d8af36981770fe4553209892 (commit)
from 03eee1573c7301b038306d6b6a1272c710317ef4 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
commit 18f6483ffe8d6191bce55c464070f9f5c08b9b01
Author: Lucas Di Pentima <lucas at curoverse.com>
Date: Mon Jul 17 14:18:05 2017 -0300
5652: Readability improvements.
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas at curoverse.com>
diff --git a/sdk/python/arvados/commands/put.py b/sdk/python/arvados/commands/put.py
index b357bc9..afd9bdc 100644
--- a/sdk/python/arvados/commands/put.py
+++ b/sdk/python/arvados/commands/put.py
@@ -195,7 +195,7 @@ total data size).
_group.add_argument('--silent', action='store_true',
help="""
-Do not produce any output unless an error happens.
+Do not print any debug messages to console. (Any error messages will still be displayed.)
""")
_group = run_opts.add_mutually_exclusive_group()
diff --git a/sdk/python/tests/test_arv_get.py b/sdk/python/tests/test_arv_get.py
index eb9a71f..fc6b846 100644
--- a/sdk/python/tests/test_arv_get.py
+++ b/sdk/python/tests/test_arv_get.py
@@ -153,21 +153,24 @@ class ArvadosGetTestCase(run_test_server.TestCaseWithServers,
tmpdir = self.make_tmpdir()
# Simulate a TTY stderr
stderr = mock.MagicMock()
- stderr.isatty.return_value = True
stdout = tutil.BytesIO()
+
# Confirm that progress is written to stderr when is a tty
+ stderr.isatty.return_value = True
r = arv_get.main(['{}/bigfile.txt'.format(c.manifest_locator()),
'{}/bigfile.txt'.format(tmpdir)],
stdout, stderr)
self.assertEqual(0, r)
self.assertEqual(b'', stdout.getvalue())
self.assertTrue(stderr.write.called)
+
# Clean up and reset stderr mock
os.remove('{}/bigfile.txt'.format(tmpdir))
stderr = mock.MagicMock()
- stderr.isatty.return_value = False
stdout = tutil.BytesIO()
+
# Confirm that progress is not written to stderr when isn't a tty
+ stderr.isatty.return_value = False
r = arv_get.main(['{}/bigfile.txt'.format(c.manifest_locator()),
'{}/bigfile.txt'.format(tmpdir)],
stdout, stderr)
diff --git a/sdk/python/tests/test_arv_put.py b/sdk/python/tests/test_arv_put.py
index 756e378..ce66181 100644
--- a/sdk/python/tests/test_arv_put.py
+++ b/sdk/python/tests/test_arv_put.py
@@ -969,7 +969,7 @@ class ArvPutIntegrationTest(run_test_server.TestCaseWithServers,
r'^\./%s.*:file2.txt' % os.path.basename(tmpdir))
self.assertRegex(c['manifest_text'], r'^.*:file3.txt')
- def test_silent_mode(self):
+ def test_silent_mode_no_errors(self):
self.authorize_with('active')
tmpdir = self.make_tmpdir()
with open(os.path.join(tmpdir, 'test.txt'), 'w') as f:
@@ -979,7 +979,7 @@ class ArvPutIntegrationTest(run_test_server.TestCaseWithServers,
stdin=subprocess.PIPE, stdout=subprocess.PIPE,
stderr=subprocess.PIPE, env=self.ENVIRON)
stdout, stderr = pipe.communicate()
- # No output should occur on normal operations
+ # No console output should occur on normal operations
self.assertNotRegex(stderr.decode(), r'.+')
self.assertNotRegex(stdout.decode(), r'.+')
commit 85bd5f3272cfc8ba8e54f4f47ea9865d7de62779
Merge: 03eee15 3a5b003
Author: Lucas Di Pentima <lucas at curoverse.com>
Date: Mon Jul 17 14:05:48 2017 -0300
5652: Merge branch 'master' into 5652-arvputget-silent-mode
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas at curoverse.com>
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list