[ARVADOS] updated: a2d956a99bb08ff983b667c8af7d528aa0e8d5ba

[git at public.curoverse.com]

Summary of changes:
 doc/api/schema/Job.html.textile.liquid    |  1 +
 sdk/cli/bin/crunch-job                    |  7 ++-
 sdk/python/arvados/api.py                 | 67 ++++++++++++++++-------
 sdk/python/tests/test_api.py              | 88 ++++++++++++++++++++++++++-----
 services/api/lib/crunch_dispatch.rb       | 26 ++++++---
 services/datamanager/keep/keep.go         |  4 +-
 services/datamanager/summary/pull_list.go |  4 +-
 7 files changed, 152 insertions(+), 45 deletions(-)

       via  a2d956a99bb08ff983b667c8af7d528aa0e8d5ba (commit)
       via  86ddb15b162a0c29b6caa4370e5cedf27bbdd69a (commit)
       via  96a3d13e88c67f6beef17877f15d97a03c63b525 (commit)
       via  8a0eb69984a93852ec888cd3e02b778b0be758ed (commit)
       via  fc5257c18b24ab0e28b248655dcabfafe9665bf3 (commit)
       via  44995fc2895a304737e324ea05f7e75e87f1458c (commit)
       via  8788c145b860e19a1f04c4dc6abdcda14c859403 (commit)
       via  dcab7519db659e2487db893fda2c8237203c3294 (commit)
       via  86dfb4312bf8fa5c7a4a79f0de1d6f0b29848252 (commit)
       via  292b28f183d7537c58d0013211760464477fbd6a (commit)
      from  5251261a5fbf8ff746f8a2ff2226d8af6ff6c12d (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 a2d956a99bb08ff983b667c8af7d528aa0e8d5ba
Author: radhika <radhika at curoverse.com>
Date:   Wed Dec 2 11:18:51 2015 -0500

    7748: use local variables for arvLogger.Update

diff --git a/services/datamanager/keep/keep.go b/services/datamanager/keep/keep.go
index cbc4047..1864383 100644
--- a/services/datamanager/keep/keep.go
+++ b/services/datamanager/keep/keep.go
@@ -459,9 +459,11 @@ func SendTrashLists(arvLogger *logger.Logger, kc *keepclient.KeepClient, spl map
 		// We need a local variable because Update doesn't call our mutator func until later,
 		// when our list variable might have been reused by the next loop iteration.
 		if arvLogger != nil {
+			url := url
+			trashLen := len(v)
 			arvLogger.Update(func(p map[string]interface{}, e map[string]interface{}) {
 				trashListInfo := logger.GetOrCreateMap(p, "trash_list_len")
-				trashListInfo[url] = len(v)
+				trashListInfo[url] = trashLen
 			})
 		}
 
diff --git a/services/datamanager/summary/pull_list.go b/services/datamanager/summary/pull_list.go
index 34149e8..6788576 100644
--- a/services/datamanager/summary/pull_list.go
+++ b/services/datamanager/summary/pull_list.go
@@ -184,8 +184,10 @@ func WritePullLists(arvLogger *logger.Logger,
 			// We need a local variable because Update doesn't call our mutator func until later,
 			// when our list variable might have been reused by the next loop iteration.
 			arvLogger.Update(func(p map[string]interface{}, e map[string]interface{}) {
+				host := host
+				listLen := len(list)
 				pullListInfo := logger.GetOrCreateMap(p, "pull_list_len")
-				pullListInfo[host] = len(list)
+				pullListInfo[host] = listLen
 			})
 		}
 

commit 86ddb15b162a0c29b6caa4370e5cedf27bbdd69a
Merge: 5251261 96a3d13
Author: radhika <radhika at curoverse.com>
Date:   Wed Dec 2 09:52:27 2015 -0500

    Merge branch 'master' into 7748-datamanager-dry-run


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


hooks/post-receive
--