[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