X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/3f8336875d5938afb6b00289e9d6c9941456b57f..bfdecdcaf7dbeabfacc0efefb864e0024dbef9ab:/services/api/app/models/pipeline_instance.rb diff --git a/services/api/app/models/pipeline_instance.rb b/services/api/app/models/pipeline_instance.rb index 651f36d30f..271b155aaf 100644 --- a/services/api/app/models/pipeline_instance.rb +++ b/services/api/app/models/pipeline_instance.rb @@ -1,3 +1,7 @@ +# Copyright (C) The Arvados Authors. All rights reserved. +# +# SPDX-License-Identifier: AGPL-3.0 + class PipelineInstance < ArvadosModel include HasUuid include KindAndEtag @@ -13,6 +17,8 @@ class PipelineInstance < ArvadosModel before_validation :update_timestamps_when_state_changes before_create :set_state_before_save before_save :set_state_before_save + before_create :create_disabled + before_update :update_disabled api_accessible :user, extend: :common do |t| t.add :pipeline_template_uuid @@ -38,6 +44,10 @@ class PipelineInstance < ArvadosModel (Complete = 'Complete'), ] + def self.limit_index_columns_read + ["components"] + end + # if all components have input, the pipeline is Ready def components_look_ready? if !self.components || self.components.empty? @@ -100,24 +110,8 @@ class PipelineInstance < ArvadosModel self.where("state = 'RunningOnServer'") end - def cancel cascade=nil - if self.state.in?([RunningOnServer, RunningOnClient]) - self.state = Paused - self.save! - elsif self.state != Paused - raise InvalidStateTransitionError - end - - return if !cascade - - # cancel all child jobs - children = self.components.andand.collect{|_, c| c['job']}.compact.collect{|j| j['uuid']}.compact - - return if children.empty? - - Job.where(uuid: children).each do |job| - job.cancel cascade if job.state.in?([Job::Queued, Job::Running]) - end + def cancel(cascade: false, need_transaction: true) + raise "No longer supported" end protected @@ -143,11 +137,9 @@ class PipelineInstance < ArvadosModel end end - if self.state.in?(States) - true - else + if !self.state.in?(States) errors.add :state, "'#{state.inspect} must be one of: [#{States.join ', '}]" - false + throw(:abort) end end @@ -170,4 +162,12 @@ class PipelineInstance < ArvadosModel end end + + def create_disabled + raise "Disabled" + end + + def update_disabled + raise "Disabled" + end end