diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-09-25 16:59:05 +0200 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-09-25 17:04:15 +0200 |
commit | 9776dbda4f6d8f0f7e9a32185cc0d493f9fe3c02 (patch) | |
tree | e8c2096da465e6323e5ba07f00398a55ed26ec1d | |
parent | 8f47d484dab12df982655c3c05305bce7624914d (diff) | |
download | gitlab-ce-9776dbda4f6d8f0f7e9a32185cc0d493f9fe3c02.tar.gz |
Use tag/branch methods to check if pipeline ref exists
-rw-r--r-- | app/services/ci/create_pipeline_service.rb | 6 | ||||
-rw-r--r-- | lib/gitlab/ci/pipeline/chain/validate.rb | 10 |
2 files changed, 5 insertions, 11 deletions
diff --git a/app/services/ci/create_pipeline_service.rb b/app/services/ci/create_pipeline_service.rb index 1a5bf7142e0..f96c4741ba4 100644 --- a/app/services/ci/create_pipeline_service.rb +++ b/app/services/ci/create_pipeline_service.rb @@ -107,8 +107,7 @@ module Ci def tag? return @is_tag if defined?(@is_tag) - @is_tag = - project.repository.ref_exists?(Gitlab::Git::TAG_REF_PREFIX + ref) + @is_tag = project.repository.tag_exists?(ref) end def ref @@ -120,7 +119,8 @@ module Ci end def pipeline_created_counter - @pipeline_created_counter ||= Gitlab::Metrics.counter(:pipelines_created_total, "Counter of pipelines created") + @pipeline_created_counter ||= Gitlab::Metrics + .counter(:pipelines_created_total, "Counter of pipelines created") end end end diff --git a/lib/gitlab/ci/pipeline/chain/validate.rb b/lib/gitlab/ci/pipeline/chain/validate.rb index e7109425d6c..4c4c8b27be6 100644 --- a/lib/gitlab/ci/pipeline/chain/validate.rb +++ b/lib/gitlab/ci/pipeline/chain/validate.rb @@ -79,22 +79,16 @@ module Gitlab end end - ## TODO, move to Pipeline as `branch_exists?` - # def branch? return @is_branch if defined?(@is_branch) - @is_branch = project.repository - .ref_exists?(Gitlab::Git::BRANCH_REF_PREFIX + pipeline.ref) + @is_branch = project.repository.branch_exists?(pipeline.ref) end - ## TODO, move to pipeline as `tag_exists?` - # def tag? return @is_tag if defined?(@is_tag) - @is_tag = project.repository - .ref_exists?(Gitlab::Git::TAG_REF_PREFIX + pipeline.ref) + @is_tag = project.repository.tag_exists?(pipeline.ref) end def error(message) |