X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/5624fec61db977d386ce03ca333241c74ca251b5..d89876219e668a3a97a6c61f92320bad0c0527c8:/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 75903ca7d0..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? @@ -101,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 @@ -150,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 @@ -177,4 +162,12 @@ class PipelineInstance < ArvadosModel end end + + def create_disabled + raise "Disabled" + end + + def update_disabled + raise "Disabled" + end end