[ARVADOS] updated: 1.3.0-2989-gb000158ff
Git user
git at public.arvados.org
Mon Aug 24 15:50:02 UTC 2020
Summary of changes:
services/api/app/models/container_request.rb | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
via b000158ff1d8083d3179614e99f4a26bf66eeae9 (commit)
from d94f8f7d99dc2ea08db259e150269a2be7396ba5 (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 b000158ff1d8083d3179614e99f4a26bf66eeae9
Author: Lucas Di Pentima <lucas at di-pentima.com.ar>
Date: Mon Aug 24 12:48:53 2020 -0300
Reverts previously dropped commit that got reintroduced by a rebase.
Refs #16470
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas at di-pentima.com.ar>
diff --git a/services/api/app/models/container_request.rb b/services/api/app/models/container_request.rb
index 16b497321..77536eee4 100644
--- a/services/api/app/models/container_request.rb
+++ b/services/api/app/models/container_request.rb
@@ -123,10 +123,10 @@ class ContainerRequest < ArvadosModel
while true
# get container lock first, then lock current container request
# (same order as Container#handle_completed). Locking always
- # requires explicit Container and ContainerRequest records reload.
+ # reloads the Container and ContainerRequest records.
c = Container.find_by_uuid(container_uuid)
- c.reload.lock! if !c.nil?
- self.reload.lock!
+ c.lock! if !c.nil?
+ self.lock!
if !c.nil? && container_uuid != c.uuid
# After locking, we've noticed a race, the container_uuid is
@@ -263,7 +263,7 @@ class ContainerRequest < ArvadosModel
if state_changed? and state == Committed and container_uuid.nil?
while true
c = Container.resolve(self)
- c.reload.lock!
+ c.lock!
if c.state == Container::Cancelled
# Lost a race, we have a lock on the container but the
# container was cancelled in a different request, restart
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list