[ARVADOS] updated: 1.3.0-2866-gf4cb214f1
Git user
git at public.arvados.org
Thu Aug 6 12:23:42 UTC 2020
Summary of changes:
services/api/app/controllers/arvados/v1/containers_controller.rb | 2 +-
services/api/config/application.rb | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
via f4cb214f12727b2145f4b9e62a7928c069b0a742 (commit)
from 7f45a8a8128471a162068a1552f66801a24ec5a8 (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 f4cb214f12727b2145f4b9e62a7928c069b0a742
Author: Lucas Di Pentima <lucas at di-pentima.com.ar>
Date: Thu Aug 6 09:22:28 2020 -0300
16470: Don't explicitly reload before locking on CR updates.
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas at di-pentima.com.ar>
diff --git a/services/api/app/controllers/arvados/v1/containers_controller.rb b/services/api/app/controllers/arvados/v1/containers_controller.rb
index b2324a571..041f55947 100644
--- a/services/api/app/controllers/arvados/v1/containers_controller.rb
+++ b/services/api/app/controllers/arvados/v1/containers_controller.rb
@@ -29,7 +29,7 @@ class Arvados::V1::ContainersController < ApplicationController
end
def update
- @object.reload.with_lock do
+ @object.with_lock do
super
end
end
diff --git a/services/api/config/application.rb b/services/api/config/application.rb
index 6bffc9a63..369294e8a 100644
--- a/services/api/config/application.rb
+++ b/services/api/config/application.rb
@@ -9,14 +9,14 @@ require "rails"
require "active_model/railtie"
require "active_job/railtie"
require "active_record/railtie"
-# require "active_storage/engine"
require "action_controller/railtie"
require "action_mailer/railtie"
require "action_view/railtie"
-# Skip ActionCable (new in Rails 5.0) as it adds '/cable' routes that we're not using
-# require "action_cable/engine"
require "sprockets/railtie"
require "rails/test_unit/railtie"
+# Skipping the following:
+# * ActionCable (new in Rails 5.0) as it adds '/cable' routes that we're not using
+# * Skip ActiveStorage (new in Rails 5.1)
require 'digest'
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list