16470: Fixes the last deprecation warnings.
authorLucas Di Pentima <lucas@di-pentima.com.ar>
Fri, 24 Jul 2020 20:12:27 +0000 (17:12 -0300)
committerLucas Di Pentima <lucas@di-pentima.com.ar>
Mon, 24 Aug 2020 15:21:27 +0000 (12:21 -0300)
commitd6897f2bee374d4a71fb96f0a794fba598174407
treeaa082d8089cf25ba2baca2add367525f04a74e06
parent29e7c39df5bbe3eae5007b0e56eb93aace7728bd
16470: Fixes the last deprecation warnings.

It turned out that .lock!() is an ActiveRecord method, and I was confusing it
with Container::lock and ContainerRequest::lock.
Both lock! and with_lock methods need the record to be persisted or reloaded
explicitly.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas@di-pentima.com.ar>
services/api/app/models/container_request.rb