[ARVADOS] updated: 741bf44b15569a3a4b1a4705b62e4f50c8365bc9

git at public.curoverse.com git at public.curoverse.com
Wed Sep 10 17:04:52 EDT 2014


Summary of changes:
 sdk/python/arvados/collection.py | 3 +++
 1 file changed, 3 insertions(+)

       via  741bf44b15569a3a4b1a4705b62e4f50c8365bc9 (commit)
      from  4ef537243058616754efde56438a193626556bca (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 741bf44b15569a3a4b1a4705b62e4f50c8365bc9
Author: Brett Smith <brett at curoverse.com>
Date:   Wed Sep 10 17:00:21 2014 -0400

    Prevent PySDK CollectionReader from sending UUIDs to Keep.
    
    No issue #.  I tickled this while I was working on #3147.  Ward saw
    the apparent symptom come up in a Job log, so I'm pushing the fix at
    his request.  I have a test prepared in my branch, along with a bunch
    of test infrastructure.

diff --git a/sdk/python/arvados/collection.py b/sdk/python/arvados/collection.py
index a06a17f..686d694 100644
--- a/sdk/python/arvados/collection.py
+++ b/sdk/python/arvados/collection.py
@@ -135,6 +135,9 @@ class CollectionReader(object):
                     uuid=self._manifest_locator).execute()
                 self._manifest_text = c['manifest_text']
             except Exception as e:
+                if not util.portable_data_hash_pattern.match(
+                      self._manifest_locator):
+                    raise
                 _logger.warning("API lookup failed for collection %s (%s: %s)",
                                 self._manifest_locator, type(e), str(e))
                 if self._keep_client is None:

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list