From: Tom Clegg <tclegg@veritasgenetics.com>
Date: Fri, 18 Aug 2017 14:37:02 +0000 (-0400)
Subject: 11309: Fix NewSummarizer()'s ability to accept a loaded API record.
X-Git-Tag: 1.1.0~76^2
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/d5491668bc4bcb7d24d881430ee712126d6c7b4e?ds=sidebyside

11309: Fix NewSummarizer()'s ability to accept a loaded API record.

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg@veritasgenetics.com>
---

diff --git a/tools/crunchstat-summary/crunchstat_summary/summarizer.py b/tools/crunchstat-summary/crunchstat_summary/summarizer.py
index 68fe2b4885..33b9230530 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())