7709: Do not set job_readable when it won't be used.
authorTom Clegg <tom@curoverse.com>
Tue, 21 Mar 2017 16:49:06 +0000 (12:49 -0400)
committerTom Clegg <tom@curoverse.com>
Thu, 30 Mar 2017 14:45:04 +0000 (10:45 -0400)
services/api/app/controllers/arvados/v1/nodes_controller.rb

index 5e2404e62c4db63360ddf91f5a6f1c801c763ce3..023d2ff888a8d2f15818c90e62388c7ab183a713 100644 (file)
@@ -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