diff options
author | Shinya Maeda <shinya@gitlab.com> | 2018-11-07 14:29:16 +0900 |
---|---|---|
committer | Shinya Maeda <shinya@gitlab.com> | 2018-11-07 14:29:16 +0900 |
commit | 9c811566f85d18bc9eb4a85c6a343cf1bfa4fbd2 (patch) | |
tree | 0d5de6f22917fa5def4e8a9a918bf0bcd5f1d3e4 /app/models/deployment.rb | |
parent | 4eeb51dc71062fa2eac828c6030388d354179efc (diff) | |
download | gitlab-ce-9c811566f85d18bc9eb4a85c6a343cf1bfa4fbd2.tar.gz |
Revert add action column changes
Diffstat (limited to 'app/models/deployment.rb')
-rw-r--r-- | app/models/deployment.rb | 21 |
1 files changed, 4 insertions, 17 deletions
diff --git a/app/models/deployment.rb b/app/models/deployment.rb index 322b3139759..305922611a6 100644 --- a/app/models/deployment.rb +++ b/app/models/deployment.rb @@ -10,7 +10,9 @@ class Deployment < ActiveRecord::Base belongs_to :user belongs_to :deployable, polymorphic: true # rubocop:disable Cop/PolymorphicAssociations - has_internal_id :iid, scope: :project, init: ->(s) { s&.project&.deployments&.maximum(:iid) } + has_internal_id :iid, scope: :project, init: ->(s) do + Deployment.where(project: s.project).maximum(:iid) if s&.project + end validates :sha, presence: true validates :ref, presence: true @@ -18,8 +20,6 @@ class Deployment < ActiveRecord::Base delegate :name, to: :environment, prefix: true scope :for_environment, -> (environment) { where(environment_id: environment) } - scope :deployed, -> { success.start } - scope :stopped, -> { success.stop } state_machine :status, initial: :created do event :run do @@ -57,11 +57,6 @@ class Deployment < ActiveRecord::Base canceled: 4 } - enum action: { - start: 1, - stop: 2 - } - def self.last_for_environment(environment) ids = self .for_environment(environment) @@ -137,7 +132,7 @@ class Deployment < ActiveRecord::Base def previous_deployment @previous_deployment ||= - project.deployments.deployed.joins(:environment) + project.deployments.success.joins(:environment) .where(environments: { name: self.environment.name }, ref: self.ref) .where.not(id: self.id) .take @@ -182,14 +177,6 @@ class Deployment < ActiveRecord::Base metrics&.merge(deployment_time: finished_at.to_i) || {} end - def deployed? - success? && start? - end - - def stopped? - success? && stop? - end - private def prometheus_adapter |