summaryrefslogtreecommitdiff
path: root/app/models/deployment.rb
diff options
context:
space:
mode:
authorShinya Maeda <shinya@gitlab.com>2018-11-07 14:29:16 +0900
committerShinya Maeda <shinya@gitlab.com>2018-11-07 14:29:16 +0900
commit9c811566f85d18bc9eb4a85c6a343cf1bfa4fbd2 (patch)
tree0d5de6f22917fa5def4e8a9a918bf0bcd5f1d3e4 /app/models/deployment.rb
parent4eeb51dc71062fa2eac828c6030388d354179efc (diff)
downloadgitlab-ce-9c811566f85d18bc9eb4a85c6a343cf1bfa4fbd2.tar.gz
Revert add action column changes
Diffstat (limited to 'app/models/deployment.rb')
-rw-r--r--app/models/deployment.rb21
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