X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/0eb72b526bf8bbb011551ecf019f604e17a534f1..65705937e9c86a4ee557a32b776745e4ae8e88db:/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 55efa0ae85..271b155aaf 100644 --- a/services/api/app/models/pipeline_instance.rb +++ b/services/api/app/models/pipeline_instance.rb @@ -17,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 @@ -109,30 +111,7 @@ class PipelineInstance < ArvadosModel end def cancel(cascade: false, need_transaction: true) - if need_transaction - ActiveRecord::Base.transaction do - cancel(cascade: cascade, need_transaction: false) - end - return - end - - 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, state: [Job::Queued, Job::Running]).each do |job| - job.cancel(cascade: cascade, need_transaction: false) - end + raise "No longer supported" end protected @@ -158,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 @@ -185,4 +162,12 @@ class PipelineInstance < ArvadosModel end end + + def create_disabled + raise "Disabled" + end + + def update_disabled + raise "Disabled" + end end