summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZ.J. van de Weg <zegerjan@gitlab.com>2016-10-19 14:49:09 +0200
committerblackst0ne <blackst0ne.ru@gmail.com>2016-10-21 08:43:36 +1100
commit8979567e439c624145ffef7ebc255b7936b5d05e (patch)
tree1a0d5e073b38bbe32fb0000fe65f51a3f08448b6
parent25d00ea871a970fb82d79067d97c84fdcb5264c5 (diff)
downloadgitlab-ce-8979567e439c624145ffef7ebc255b7936b5d05e.tar.gz
Use deployment IID when saving refs
-rw-r--r--app/models/deployment.rb2
-rw-r--r--app/models/environment.rb4
-rw-r--r--spec/controllers/projects/merge_requests_controller_spec.rb2
3 files changed, 4 insertions, 4 deletions
diff --git a/app/models/deployment.rb b/app/models/deployment.rb
index 1f8c5fb3d85..c843903877b 100644
--- a/app/models/deployment.rb
+++ b/app/models/deployment.rb
@@ -102,6 +102,6 @@ class Deployment < ActiveRecord::Base
private
def ref_path
- File.join(environment.ref_path, 'deployments', id.to_s)
+ File.join(environment.ref_path, 'deployments', iid.to_s)
end
end
diff --git a/app/models/environment.rb b/app/models/environment.rb
index d575f1dc73a..73f415c0ef0 100644
--- a/app/models/environment.rb
+++ b/app/models/environment.rb
@@ -71,8 +71,8 @@ class Environment < ActiveRecord::Base
return nil unless ref
- deployment_id = ref.split('/').last
- deployments.find(deployment_id)
+ deployment_iid = ref.split('/').last
+ deployments.find_by(iid: deployment_iid)
end
def ref_path
diff --git a/spec/controllers/projects/merge_requests_controller_spec.rb b/spec/controllers/projects/merge_requests_controller_spec.rb
index d6980471ea4..940d54f8686 100644
--- a/spec/controllers/projects/merge_requests_controller_spec.rb
+++ b/spec/controllers/projects/merge_requests_controller_spec.rb
@@ -913,7 +913,7 @@ describe Projects::MergeRequestsController do
end
describe 'GET ci_environments_status' do
- context 'when the environment is from a forked project' do
+ context 'the environment is from a forked project' do
let!(:forked) { create(:project) }
let!(:environment) { create(:environment, project: forked) }
let!(:deployment) { create(:deployment, environment: environment, sha: forked.commit.id, ref: 'master') }