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>
Fri, 24 Jul 2020 20:12:27 +0000 (17:12 -0300)
commit571e9ea468feaa22ccf56b9585a36d6d84f1b8bd
treeb8268ba56026f05154de070467488f78414a1ae2
parent070bcb4e6afda609a371a5bfc7d0927f379e4531
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