From ce5f1a58b22bcf19bcedfb5d8602290d4a3025ff Mon Sep 17 00:00:00 2001 From: Tom Clegg Date: Wed, 14 Feb 2018 10:10:24 -0500 Subject: [PATCH] 12199: Fix job_uuid missing from nodes#update response. Arvados-DCO-1.1-Signed-off-by: Tom Clegg --- .../api/app/controllers/arvados/v1/nodes_controller.rb | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/services/api/app/controllers/arvados/v1/nodes_controller.rb b/services/api/app/controllers/arvados/v1/nodes_controller.rb index 73f1dee54a..247d15e1c1 100644 --- a/services/api/app/controllers/arvados/v1/nodes_controller.rb +++ b/services/api/app/controllers/arvados/v1/nodes_controller.rb @@ -9,13 +9,6 @@ class Arvados::V1::NodesController < ApplicationController include DbCurrentTime - def update - if resource_attrs[:job_uuid].is_a? String - @object.job_readable = readable_job_uuids([resource_attrs[:job_uuid]]).any? - end - super - end - def self._ping_requires_parameters { ping_secret: {required: true} } end @@ -38,6 +31,9 @@ class Arvados::V1::NodesController < ApplicationController end def update + if resource_attrs[:job_uuid].is_a? String + @object.job_readable = readable_job_uuids([resource_attrs[:job_uuid]]).any? + end attrs_to_update = resource_attrs.reject { |k,v| [:kind, :etag, :href].index k } -- 2.30.2