From: Tom Clegg <tom@curoverse.com>
Date: Thu, 20 Apr 2017 02:09:45 +0000 (-0400)
Subject: 11470: Fix accidentally cleared fields during job_task update.
X-Git-Tag: 1.1.0~293^2~1
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/3c34e713aa343d56c7cea00a9c998b06dbf411d6?hp=82c40aa7d30cd8e68e2a1bdc0bd8bf03cdfea029

11470: Fix accidentally cleared fields during job_task update.

Updating with a Task struct had the effect of clearing job_uuid,
created_by_job_task_uuid, and parameters fields, and resetting
sequence to 0.
---

diff --git a/sdk/go/crunchrunner/crunchrunner.go b/sdk/go/crunchrunner/crunchrunner.go
index 5d7e10be4b..36343d3df3 100644
--- a/sdk/go/crunchrunner/crunchrunner.go
+++ b/sdk/go/crunchrunner/crunchrunner.go
@@ -254,10 +254,10 @@ func runner(api IArvadosClient,
 			}
 			err = api.Update("job_tasks", taskUuid,
 				map[string]interface{}{
-					"job_task": Task{
-						Output:   "",
-						Success:  true,
-						Progress: 1.0}},
+					"job_task": map[string]interface{}{
+						"output":   "",
+						"success":  true,
+						"progress": 1.0}},
 				nil)
 			return nil
 		}