services/api/app/models/container.rb

   1 # Copyright (C) The Arvados Authors. All rights reserved.
   2 #
   3 # SPDX-License-Identifier: AGPL-3.0
   4
   5 require 'log_reuse_info'
   6 require 'whitelist_update'
   7 require 'safe_json'
   8 require 'update_priority'
   9
  10 class Container < ArvadosModel
  11   include ArvadosModelUpdates
  12   include HasUuid
  13   include KindAndEtag
  14   include CommonApiTemplate
  15   include WhitelistUpdate
  16   extend CurrentApiClient
  17   extend DbCurrentTime
  18   extend LogReuseInfo
  19
  20   serialize :environment, Hash
  21   serialize :mounts, Hash
  22   serialize :runtime_constraints, Hash
  23   serialize :command, Array
  24   serialize :scheduling_parameters, Hash
  25   serialize :secret_mounts, Hash
  26   serialize :runtime_status, Hash
  27
  28   before_validation :fill_field_defaults, :if => :new_record?
  29   before_validation :set_timestamps
  30   validates :command, :container_image, :output_path, :cwd, :priority, { presence: true }
  31   validates :priority, numericality: { only_integer: true, greater_than_or_equal_to: 0 }
  32   validate :validate_runtime_status
  33   validate :validate_state_change
  34   validate :validate_change
  35   validate :validate_lock
  36   validate :validate_output
  37   after_validation :assign_auth
  38   before_save :sort_serialized_attrs
  39   before_save :update_secret_mounts_md5
  40   before_save :scrub_secrets
  41   before_save :clear_runtime_status_when_queued
  42   after_save :update_cr_logs
  43   after_save :handle_completed
  44   after_save :propagate_priority
  45   after_commit { UpdatePriority.run_update_thread }
  46
  47   has_many :container_requests, :foreign_key => :container_uuid, :class_name => 'ContainerRequest', :primary_key => :uuid
  48   belongs_to :auth, :class_name => 'ApiClientAuthorization', :foreign_key => :auth_uuid, :primary_key => :uuid
  49
  50   api_accessible :user, extend: :common do |t|
  51     t.add :command
  52     t.add :container_image
  53     t.add :cwd
  54     t.add :environment
  55     t.add :exit_code
  56     t.add :finished_at
  57     t.add :locked_by_uuid
  58     t.add :log
  59     t.add :mounts
  60     t.add :output
  61     t.add :output_path
  62     t.add :priority
  63     t.add :progress
  64     t.add :runtime_constraints
  65     t.add :runtime_status
  66     t.add :started_at
  67     t.add :state
  68     t.add :auth_uuid
  69     t.add :scheduling_parameters
  70     t.add :runtime_user_uuid
  71     t.add :runtime_auth_scopes
  72   end
  73
  74   # Supported states for a container
  75   States =
  76     [
  77      (Queued = 'Queued'),
  78      (Locked = 'Locked'),
  79      (Running = 'Running'),
  80      (Complete = 'Complete'),
  81      (Cancelled = 'Cancelled')
  82     ]
  83
  84   State_transitions = {
  85     nil => [Queued],
  86     Queued => [Locked, Cancelled],
  87     Locked => [Queued, Running, Cancelled],
  88     Running => [Complete, Cancelled]
  89   }
  90
  91   def self.limit_index_columns_read
  92     ["mounts"]
  93   end
  94
  95   def self.full_text_searchable_columns
  96     super - ["secret_mounts", "secret_mounts_md5", "runtime_token"]
  97   end
  98
  99   def self.searchable_columns *args
 100     super - ["secret_mounts_md5", "runtime_token"]
 101   end
 102
 103   def logged_attributes
 104     super.except('secret_mounts', 'runtime_token')
 105   end
 106
 107   def state_transitions
 108     State_transitions
 109   end
 110
 111   # Container priority is the highest "computed priority" of any
 112   # matching request. The computed priority of a container-submitted
 113   # request is the priority of the submitting container. The computed
 114   # priority of a user-submitted request is a function of
 115   # user-assigned priority and request creation time.
 116   def update_priority!
 117     return if ![Queued, Locked, Running].include?(state)
 118     p = ContainerRequest.
 119         where('container_uuid=? and priority>0', uuid).
 120         includes(:requesting_container).
 121         lock(true).
 122         map do |cr|
 123       if cr.requesting_container
 124         cr.requesting_container.priority
 125       else
 126         (cr.priority << 50) - (cr.created_at.to_time.to_f * 1000).to_i
 127       end
 128     end.max || 0
 129     update_attributes!(priority: p)
 130   end
 131
 132   def propagate_priority
 133     return true unless priority_changed?
 134     act_as_system_user do
 135       # Update the priority of child container requests to match new
 136       # priority of the parent container (ignoring requests with no
 137       # container assigned, because their priority doesn't matter).
 138       ContainerRequest.
 139         where(requesting_container_uuid: self.uuid,
 140               state: ContainerRequest::Committed).
 141         where('container_uuid is not null').
 142         includes(:container).
 143         map(&:container).
 144         map(&:update_priority!)
 145     end
 146   end
 147
 148   # Create a new container (or find an existing one) to satisfy the
 149   # given container request.
 150   def self.resolve(req)
 151     if req.runtime_token.nil?
 152       runtime_user = if req.modified_by_user_uuid.nil?
 153                        current_user
 154                      else
 155                        User.find_by_uuid(req.modified_by_user_uuid)
 156                      end
 157       runtime_auth_scopes = ["all"]
 158     else
 159       auth = ApiClientAuthorization.validate(token: req.runtime_token)
 160       if auth.nil?
 161         raise ArgumentError.new "Invalid runtime token"
 162       end
 163       runtime_user = User.find_by_id(auth.user_id)
 164       runtime_auth_scopes = auth.scopes
 165     end
 166     c_attrs = act_as_user runtime_user do
 167       {
 168         command: req.command,
 169         cwd: req.cwd,
 170         environment: req.environment,
 171         output_path: req.output_path,
 172         container_image: resolve_container_image(req.container_image),
 173         mounts: resolve_mounts(req.mounts),
 174         runtime_constraints: resolve_runtime_constraints(req.runtime_constraints),
 175         scheduling_parameters: req.scheduling_parameters,
 176         secret_mounts: req.secret_mounts,
 177         runtime_token: req.runtime_token,
 178         runtime_user_uuid: runtime_user.uuid,
 179         runtime_auth_scopes: runtime_auth_scopes
 180       }
 181     end
 182     act_as_system_user do
 183       if req.use_existing && (reusable = find_reusable(c_attrs))
 184         reusable
 185       else
 186         Container.create!(c_attrs)
 187       end
 188     end
 189   end
 190
 191   # Return a runtime_constraints hash that complies with requested but
 192   # is suitable for saving in a container record, i.e., has specific
 193   # values instead of ranges.
 194   #
 195   # Doing this as a step separate from other resolutions, like "git
 196   # revision range to commit hash", makes sense only when there is no
 197   # opportunity to reuse an existing container (e.g., container reuse
 198   # is not implemented yet, or we have already found that no existing
 199   # containers are suitable).
 200   def self.resolve_runtime_constraints(runtime_constraints)
 201     rc = {}
 202     defaults = {
 203       'keep_cache_ram' =>
 204       Rails.configuration.container_default_keep_cache_ram,
 205     }
 206     defaults.merge(runtime_constraints).each do |k, v|
 207       if v.is_a? Array
 208         rc[k] = v[0]
 209       else
 210         rc[k] = v
 211       end
 212     end
 213     rc
 214   end
 215
 216   # Return a mounts hash suitable for a Container, i.e., with every
 217   # readonly collection UUID resolved to a PDH.
 218   def self.resolve_mounts(mounts)
 219     c_mounts = {}
 220     mounts.each do |k, mount|
 221       mount = mount.dup
 222       c_mounts[k] = mount
 223       if mount['kind'] != 'collection'
 224         next
 225       end
 226
 227       uuid = mount.delete 'uuid'
 228
 229       if mount['portable_data_hash'].nil? and !uuid.nil?
 230         # PDH not supplied, try by UUID
 231         c = Collection.
 232           readable_by(current_user).
 233           where(uuid: uuid).
 234           select(:portable_data_hash).
 235           first
 236         if !c
 237           raise ArvadosModel::UnresolvableContainerError.new "cannot mount collection #{uuid.inspect}: not found"
 238         end
 239         mount['portable_data_hash'] = c.portable_data_hash
 240       end
 241     end
 242     return c_mounts
 243   end
 244
 245   # Return a container_image PDH suitable for a Container.
 246   def self.resolve_container_image(container_image)
 247     coll = Collection.for_latest_docker_image(container_image)
 248     if !coll
 249       raise ArvadosModel::UnresolvableContainerError.new "docker image #{container_image.inspect} not found"
 250     end
 251     coll.portable_data_hash
 252   end
 253
 254   def self.find_reusable(attrs)
 255     log_reuse_info { "starting with #{Container.all.count} container records in database" }
 256     candidates = Container.where_serialized(:command, attrs[:command], md5: true)
 257     log_reuse_info(candidates) { "after filtering on command #{attrs[:command].inspect}" }
 258
 259     candidates = candidates.where('cwd = ?', attrs[:cwd])
 260     log_reuse_info(candidates) { "after filtering on cwd #{attrs[:cwd].inspect}" }
 261
 262     candidates = candidates.where_serialized(:environment, attrs[:environment], md5: true)
 263     log_reuse_info(candidates) { "after filtering on environment #{attrs[:environment].inspect}" }
 264
 265     candidates = candidates.where('output_path = ?', attrs[:output_path])
 266     log_reuse_info(candidates) { "after filtering on output_path #{attrs[:output_path].inspect}" }
 267
 268     image = resolve_container_image(attrs[:container_image])
 269     candidates = candidates.where('container_image = ?', image)
 270     log_reuse_info(candidates) { "after filtering on container_image #{image.inspect} (resolved from #{attrs[:container_image].inspect})" }
 271
 272     candidates = candidates.where_serialized(:mounts, resolve_mounts(attrs[:mounts]), md5: true)
 273     log_reuse_info(candidates) { "after filtering on mounts #{attrs[:mounts].inspect}" }
 274
 275     secret_mounts_md5 = Digest::MD5.hexdigest(SafeJSON.dump(self.deep_sort_hash(attrs[:secret_mounts])))
 276     candidates = candidates.where('secret_mounts_md5 = ?', secret_mounts_md5)
 277     log_reuse_info(candidates) { "after filtering on secret_mounts_md5 #{secret_mounts_md5.inspect}" }
 278
 279     candidates = candidates.where_serialized(:runtime_constraints, resolve_runtime_constraints(attrs[:runtime_constraints]), md5: true)
 280     log_reuse_info(candidates) { "after filtering on runtime_constraints #{attrs[:runtime_constraints].inspect}" }
 281
 282     candidates = candidates.where('runtime_user_uuid = ? or (runtime_user_uuid is NULL and runtime_auth_scopes is NULL)',
 283                                   attrs[:runtime_user_uuid])
 284     log_reuse_info(candidates) { "after filtering on runtime_user_uuid #{attrs[:runtime_user_uuid].inspect}" }
 285
 286     candidates = candidates.where('runtime_auth_scopes = ? or (runtime_user_uuid is NULL and runtime_auth_scopes is NULL)',
 287                                   SafeJSON.dump(attrs[:runtime_auth_scopes].sort))
 288     log_reuse_info(candidates) { "after filtering on runtime_auth_scopes #{attrs[:runtime_auth_scopes].inspect}" }
 289
 290     log_reuse_info { "checking for state=Complete with readable output and log..." }
 291
 292     select_readable_pdh = Collection.
 293       readable_by(current_user).
 294       select(:portable_data_hash).
 295       to_sql
 296
 297     usable = candidates.where(state: Complete, exit_code: 0)
 298     log_reuse_info(usable) { "with state=Complete, exit_code=0" }
 299
 300     usable = usable.where("log IN (#{select_readable_pdh})")
 301     log_reuse_info(usable) { "with readable log" }
 302
 303     usable = usable.where("output IN (#{select_readable_pdh})")
 304     log_reuse_info(usable) { "with readable output" }
 305
 306     usable = usable.order('finished_at ASC').limit(1).first
 307     if usable
 308       log_reuse_info { "done, reusing container #{usable.uuid} with state=Complete" }
 309       return usable
 310     end
 311
 312     # Check for non-failing Running candidates and return the most likely to finish sooner.
 313     log_reuse_info { "checking for state=Running..." }
 314     running = candidates.where(state: Running).
 315               where("(runtime_status->'error') is null").
 316               order('progress desc, started_at asc').
 317               limit(1).first
 318     if running
 319       log_reuse_info { "done, reusing container #{running.uuid} with state=Running" }
 320       return running
 321     else
 322       log_reuse_info { "have no containers in Running state" }
 323     end
 324
 325     # Check for Locked or Queued ones and return the most likely to start first.
 326     locked_or_queued = candidates.
 327                        where("state IN (?)", [Locked, Queued]).
 328                        order('state asc, priority desc, created_at asc').
 329                        limit(1).first
 330     if locked_or_queued
 331       log_reuse_info { "done, reusing container #{locked_or_queued.uuid} with state=#{locked_or_queued.state}" }
 332       return locked_or_queued
 333     else
 334       log_reuse_info { "have no containers in Locked or Queued state" }
 335     end
 336
 337     log_reuse_info { "done, no reusable container found" }
 338     nil
 339   end
 340
 341   def check_lock_fail
 342     if self.state != Queued
 343       raise LockFailedError.new("cannot lock when #{self.state}")
 344     elsif self.priority <= 0
 345       raise LockFailedError.new("cannot lock when priority<=0")
 346     end
 347   end
 348
 349   def lock
 350     # Check invalid state transitions once before getting the lock
 351     # (because it's cheaper that way) and once after getting the lock
 352     # (because state might have changed while acquiring the lock).
 353     check_lock_fail
 354     transaction do
 355       reload
 356       check_lock_fail
 357       update_attributes!(state: Locked)
 358     end
 359   end
 360
 361   def check_unlock_fail
 362     if self.state != Locked
 363       raise InvalidStateTransitionError.new("cannot unlock when #{self.state}")
 364     elsif self.locked_by_uuid != current_api_client_authorization.uuid
 365       raise InvalidStateTransitionError.new("locked by a different token")
 366     end
 367   end
 368
 369   def unlock
 370     # Check invalid state transitions twice (see lock)
 371     check_unlock_fail
 372     transaction do
 373       reload(lock: 'FOR UPDATE')
 374       check_unlock_fail
 375       update_attributes!(state: Queued)
 376     end
 377   end
 378
 379   def self.readable_by(*users_list)
 380     # Load optional keyword arguments, if they exist.
 381     if users_list.last.is_a? Hash
 382       kwargs = users_list.pop
 383     else
 384       kwargs = {}
 385     end
 386     Container.where(ContainerRequest.readable_by(*users_list).where("containers.uuid = container_requests.container_uuid").exists)
 387   end
 388
 389   def final?
 390     [Complete, Cancelled].include?(self.state)
 391   end
 392
 393   def self.for_current_token
 394     return if !current_api_client_authorization
 395     _, _, _, container_uuid = Thread.current[:token].split('/')
 396     if container_uuid.nil?
 397       Container.where(auth_uuid: current_api_client_authorization.uuid).first
 398     else
 399       Container.where('auth_uuid=? or (uuid=? and runtime_token=?)',
 400                       current_api_client_authorization.uuid,
 401                       container_uuid,
 402                       current_api_client_authorization.token).first
 403     end
 404   end
 405
 406   protected
 407
 408   def fill_field_defaults
 409     self.state ||= Queued
 410     self.environment ||= {}
 411     self.runtime_constraints ||= {}
 412     self.mounts ||= {}
 413     self.cwd ||= "."
 414     self.priority ||= 0
 415     self.scheduling_parameters ||= {}
 416   end
 417
 418   def permission_to_create
 419     current_user.andand.is_admin
 420   end
 421
 422   def ensure_owner_uuid_is_permitted
 423     # validate_change ensures owner_uuid can't be changed at all --
 424     # except during create, which requires admin privileges. Checking
 425     # permission here would be superfluous.
 426     true
 427   end
 428
 429   def set_timestamps
 430     if self.state_changed? and self.state == Running
 431       self.started_at ||= db_current_time
 432     end
 433
 434     if self.state_changed? and [Complete, Cancelled].include? self.state
 435       self.finished_at ||= db_current_time
 436     end
 437   end
 438
 439   # Check that well-known runtime status keys have desired data types
 440   def validate_runtime_status
 441     [
 442       'error', 'errorDetail', 'warning', 'warningDetail', 'activity'
 443     ].each do |k|
 444       if self.runtime_status.andand.include?(k) && !self.runtime_status[k].is_a?(String)
 445         errors.add(:runtime_status, "'#{k}' value must be a string")
 446       end
 447     end
 448   end
 449
 450   def validate_change
 451     permitted = [:state]
 452     progress_attrs = [:progress, :runtime_status, :log, :output]
 453     final_attrs = [:exit_code, :finished_at]
 454
 455     if self.new_record?
 456       permitted.push(:owner_uuid, :command, :container_image, :cwd,
 457                      :environment, :mounts, :output_path, :priority,
 458                      :runtime_constraints, :scheduling_parameters,
 459                      :secret_mounts, :runtime_token,
 460                      :runtime_user_uuid, :runtime_auth_scopes)
 461     end
 462
 463     case self.state
 464     when Locked
 465       permitted.push :priority, :runtime_status, :log
 466
 467     when Queued
 468       permitted.push :priority
 469
 470     when Running
 471       permitted.push :priority, *progress_attrs
 472       if self.state_changed?
 473         permitted.push :started_at
 474       end
 475
 476     when Complete
 477       if self.state_was == Running
 478         permitted.push *final_attrs, *progress_attrs
 479       end
 480
 481     when Cancelled
 482       case self.state_was
 483       when Running
 484         permitted.push :finished_at, *progress_attrs
 485       when Queued, Locked
 486         permitted.push :finished_at, :log
 487       end
 488
 489     else
 490       # The state_transitions check will add an error message for this
 491       return false
 492     end
 493
 494     if self.state == Running &&
 495        !current_api_client_authorization.nil? &&
 496        (current_api_client_authorization.uuid == self.auth_uuid ||
 497         current_api_client_authorization.token == self.runtime_token)
 498       # The contained process itself can write final attrs but can't
 499       # change priority or log.
 500       permitted.push *final_attrs
 501       permitted = permitted - [:log, :priority]
 502     elsif self.locked_by_uuid && self.locked_by_uuid != current_api_client_authorization.andand.uuid
 503       # When locked, progress fields cannot be updated by the wrong
 504       # dispatcher, even though it has admin privileges.
 505       permitted = permitted - progress_attrs
 506     end
 507     check_update_whitelist permitted
 508   end
 509
 510   def validate_lock
 511     if [Locked, Running].include? self.state
 512       # If the Container was already locked, locked_by_uuid must not
 513       # changes. Otherwise, the current auth gets the lock.
 514       need_lock = locked_by_uuid_was || current_api_client_authorization.andand.uuid
 515     else
 516       need_lock = nil
 517     end
 518
 519     # The caller can provide a new value for locked_by_uuid, but only
 520     # if it's exactly what we expect. This allows a caller to perform
 521     # an update like {"state":"Unlocked","locked_by_uuid":null}.
 522     if self.locked_by_uuid_changed?
 523       if self.locked_by_uuid != need_lock
 524         return errors.add :locked_by_uuid, "can only change to #{need_lock}"
 525       end
 526     end
 527     self.locked_by_uuid = need_lock
 528   end
 529
 530   def validate_output
 531     # Output must exist and be readable by the current user.  This is so
 532     # that a container cannot "claim" a collection that it doesn't otherwise
 533     # have access to just by setting the output field to the collection PDH.
 534     if output_changed?
 535       c = Collection.
 536             readable_by(current_user, {include_trash: true}).
 537             where(portable_data_hash: self.output).
 538             first
 539       if !c
 540         errors.add :output, "collection must exist and be readable by current user."
 541       end
 542     end
 543   end
 544
 545   def update_cr_logs
 546     # If self.final?, this update is superfluous: the final log/output
 547     # update will be done when handle_completed calls finalize! on
 548     # each requesting CR.
 549     return if self.final? || !self.log_changed?
 550     leave_modified_by_user_alone do
 551       ContainerRequest.where(container_uuid: self.uuid).each do |cr|
 552         cr.update_collections(container: self, collections: ['log'])
 553         cr.save!
 554       end
 555     end
 556   end
 557
 558   def assign_auth
 559     if self.auth_uuid_changed?
 560          return errors.add :auth_uuid, 'is readonly'
 561     end
 562     if not [Locked, Running].include? self.state
 563       # don't need one
 564       self.auth.andand.update_attributes(expires_at: db_current_time)
 565       self.auth = nil
 566       return
 567     elsif self.auth
 568       # already have one
 569       return
 570     end
 571     if self.runtime_token.nil?
 572       if self.runtime_user_uuid.nil?
 573         # legacy behavior, we don't have a runtime_user_uuid so get
 574         # the user from the highest priority container request, needed
 575         # when performing an upgrade and there are queued containers,
 576         # and some tests.
 577         cr = ContainerRequest.
 578                where('container_uuid=? and priority>0', self.uuid).
 579                order('priority desc').
 580                first
 581         if !cr
 582           return errors.add :auth_uuid, "cannot be assigned because priority <= 0"
 583         end
 584         self.runtime_user_uuid = cr.modified_by_user_uuid
 585         self.runtime_auth_scopes = ["all"]
 586       end
 587
 588       # generate a new token
 589       self.auth = ApiClientAuthorization.
 590                     create!(user_id: User.find_by_uuid(self.runtime_user_uuid).id,
 591                             api_client_id: 0,
 592                             scopes: self.runtime_auth_scopes)
 593     end
 594   end
 595
 596   def sort_serialized_attrs
 597     if self.environment_changed?
 598       self.environment = self.class.deep_sort_hash(self.environment)
 599     end
 600     if self.mounts_changed?
 601       self.mounts = self.class.deep_sort_hash(self.mounts)
 602     end
 603     if self.runtime_constraints_changed?
 604       self.runtime_constraints = self.class.deep_sort_hash(self.runtime_constraints)
 605     end
 606     if self.scheduling_parameters_changed?
 607       self.scheduling_parameters = self.class.deep_sort_hash(self.scheduling_parameters)
 608     end
 609     if self.runtime_auth_scopes_changed?
 610       self.runtime_auth_scopes = self.runtime_auth_scopes.sort
 611     end
 612   end
 613
 614   def update_secret_mounts_md5
 615     if self.secret_mounts_changed?
 616       self.secret_mounts_md5 = Digest::MD5.hexdigest(
 617         SafeJSON.dump(self.class.deep_sort_hash(self.secret_mounts)))
 618     end
 619   end
 620
 621   def scrub_secrets
 622     # this runs after update_secret_mounts_md5, so the
 623     # secret_mounts_md5 will still reflect the secrets that are being
 624     # scrubbed here.
 625     if self.state_changed? && self.final?
 626       self.secret_mounts = {}
 627       self.runtime_token = nil
 628     end
 629   end
 630
 631   def clear_runtime_status_when_queued
 632     # Avoid leaking status messages between different dispatch attempts
 633     if self.state_was == Locked && self.state == Queued
 634       self.runtime_status = {}
 635     end
 636   end
 637
 638   def handle_completed
 639     # This container is finished so finalize any associated container requests
 640     # that are associated with this container.
 641     if self.state_changed? and self.final?
 642       act_as_system_user do
 643
 644         if self.state == Cancelled
 645           retryable_requests = ContainerRequest.where("container_uuid = ? and priority > 0 and state = 'Committed' and container_count < container_count_max", uuid)
 646         else
 647           retryable_requests = []
 648         end
 649
 650         if retryable_requests.any?
 651           c_attrs = {
 652             command: self.command,
 653             cwd: self.cwd,
 654             environment: self.environment,
 655             output_path: self.output_path,
 656             container_image: self.container_image,
 657             mounts: self.mounts,
 658             runtime_constraints: self.runtime_constraints,
 659             scheduling_parameters: self.scheduling_parameters,
 660             secret_mounts: self.secret_mounts_was,
 661             runtime_token: self.runtime_token_was,
 662             runtime_user_uuid: self.runtime_user_uuid,
 663             runtime_auth_scopes: self.runtime_auth_scopes
 664           }
 665           c = Container.create! c_attrs
 666           retryable_requests.each do |cr|
 667             cr.with_lock do
 668               leave_modified_by_user_alone do
 669                 # Use row locking because this increments container_count
 670                 cr.container_uuid = c.uuid
 671                 cr.save!
 672               end
 673             end
 674           end
 675         end
 676
 677         # Notify container requests associated with this container
 678         ContainerRequest.where(container_uuid: uuid,
 679                                state: ContainerRequest::Committed).each do |cr|
 680           leave_modified_by_user_alone do
 681             cr.finalize!
 682           end
 683         end
 684
 685         # Cancel outstanding container requests made by this container.
 686         ContainerRequest.
 687           includes(:container).
 688           where(requesting_container_uuid: uuid,
 689                 state: ContainerRequest::Committed).each do |cr|
 690           leave_modified_by_user_alone do
 691             cr.update_attributes!(priority: 0)
 692             cr.container.reload
 693             if cr.container.state == Container::Queued || cr.container.state == Container::Locked
 694               # If the child container hasn't started yet, finalize the
 695               # child CR now instead of leaving it "on hold", i.e.,
 696               # Queued with priority 0.  (OTOH, if the child is already
 697               # running, leave it alone so it can get cancelled the
 698               # usual way, get a copy of the log collection, etc.)
 699               cr.update_attributes!(state: ContainerRequest::Final)
 700             end
 701           end
 702         end
 703       end
 704     end
 705   end
 706 end