From: Tom Clegg <tom@curoverse.com>
Date: Tue, 21 Mar 2017 16:49:06 +0000 (-0400)
Subject: 7709: Do not set job_readable when it won't be used.
X-Git-Tag: 1.1.0~285^2~25
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/05d453ec38b10a022ea6db77867957e7115b9b35?hp=09dcf71e59907c2eaf4b94918c63da07193481a4

7709: Do not set job_readable when it won't be used.
---

diff --git a/services/api/app/controllers/arvados/v1/nodes_controller.rb b/services/api/app/controllers/arvados/v1/nodes_controller.rb
index 5e2404e62c..023d2ff888 100644
--- a/services/api/app/controllers/arvados/v1/nodes_controller.rb
+++ b/services/api/app/controllers/arvados/v1/nodes_controller.rb
@@ -46,10 +46,12 @@ class Arvados::V1::NodesController < ApplicationController
       @objects = model_class.where('last_ping_at >= ?', db_current_time - 1.hours)
     end
     super
-    job_uuids = @objects.map { |n| n[:job_uuid] }.compact
-    assoc_jobs = readable_job_uuids(job_uuids)
-    @objects.each do |node|
-      node.job_readable = assoc_jobs.include?(node[:job_uuid])
+    if @select.nil? or @select.include? 'job_uuid'
+      job_uuids = @objects.map { |n| n[:job_uuid] }.compact
+      assoc_jobs = readable_job_uuids(job_uuids)
+      @objects = @objects.each do |node|
+        node.job_readable = assoc_jobs.include?(node[:job_uuid])
+      end
     end
   end