[arvados] updated: 2.6.0-100-gd018aa211

git repository hosting git at public.arvados.org
Thu May 4 17:34:46 UTC 2023


Summary of changes:
 services/api/app/controllers/arvados/v1/containers_controller.rb | 4 ++++
 1 file changed, 4 insertions(+)

       via  d018aa211d3efc93c711248851db379b19fa60a1 (commit)
      from  db14d5a612dd8db80a97d954f3364f23f21b005d (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 d018aa211d3efc93c711248851db379b19fa60a1
Author: Peter Amstutz <peter.amstutz at curii.com>
Date:   Thu May 4 13:34:27 2023 -0400

    20472: Need to make sure :id is selected in update_priority for reload
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>

diff --git a/services/api/app/controllers/arvados/v1/containers_controller.rb b/services/api/app/controllers/arvados/v1/containers_controller.rb
index 8e36cdc0d..9247327ac 100644
--- a/services/api/app/controllers/arvados/v1/containers_controller.rb
+++ b/services/api/app/controllers/arvados/v1/containers_controller.rb
@@ -52,6 +52,10 @@ class Arvados::V1::ContainersController < ApplicationController
       # Avoid loading more fields than we need
       @objects = @objects.select(:id, :uuid, :state, :priority, :auth_uuid, :locked_by_uuid, :lock_count)
       @select = %w(uuid state priority auth_uuid locked_by_uuid)
+    elsif action_name == 'update_priority'
+      # We're going to reload in update_priority!, which will select
+      # all attributes, but will fail if we don't select :id now.
+      @objects = @objects.select(:id, :uuid)
     end
   end
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list