[ARVADOS] updated: d5491668bc4bcb7d24d881430ee712126d6c7b4e

Git user git at public.curoverse.com
Fri Aug 18 10:40:42 EDT 2017


Summary of changes:
 tools/crunchstat-summary/crunchstat_summary/summarizer.py | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

       via  d5491668bc4bcb7d24d881430ee712126d6c7b4e (commit)
      from  c77cb0091a8488eae52341920e8e32beb8fc47dc (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 d5491668bc4bcb7d24d881430ee712126d6c7b4e
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Fri Aug 18 10:37:02 2017 -0400

    11309: Fix NewSummarizer()'s ability to accept a loaded API record.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/tools/crunchstat-summary/crunchstat_summary/summarizer.py b/tools/crunchstat-summary/crunchstat_summary/summarizer.py
index 68fe2b4..33b9230 100644
--- a/tools/crunchstat-summary/crunchstat_summary/summarizer.py
+++ b/tools/crunchstat-summary/crunchstat_summary/summarizer.py
@@ -431,11 +431,14 @@ class CollectionSummarizer(Summarizer):
         self.label = collection_id
 
 
-def NewSummarizer(process, **kwargs):
+def NewSummarizer(process_or_uuid, **kwargs):
     """Construct with the appropriate subclass for this uuid/object."""
 
-    if not isinstance(process, dict):
-        uuid = process
+    if isinstance(process_or_uuid, dict):
+        process = process_or_uuid
+        uuid = process['uuid']
+    else:
+        uuid = process_or_uuid
         process = None
         arv = arvados.api('v1', model=OrderedJsonModel())
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list