Commit 04b8e00f authored by Alejandro Rodríguez's avatar Alejandro Rodríguez

Use porcelain commit lookup method on CI::CreatePipelineService

Before we were using a "plumbing" Gitlab::Git method that does not go
through Gitaly migration checking.
parent 22fc4863
...@@ -65,7 +65,7 @@ module Ci ...@@ -65,7 +65,7 @@ module Ci
project.pipelines project.pipelines
.where(ref: pipeline.ref) .where(ref: pipeline.ref)
.where.not(id: pipeline.id) .where.not(id: pipeline.id)
.where.not(sha: project.repository.sha_from_ref(pipeline.ref)) .where.not(sha: project.commit(pipeline.ref).try(:id))
.created_or_pending .created_or_pending
end end
......
---
title: Use porcelain commit lookup method on CI::CreatePipelineService
merge_request: 17911
author:
type: fixed
...@@ -516,10 +516,6 @@ module Gitlab ...@@ -516,10 +516,6 @@ module Gitlab
end end
end end
def sha_from_ref(ref)
rev_parse_target(ref).oid
end
# Return the object that +revspec+ points to. If +revspec+ is an # Return the object that +revspec+ points to. If +revspec+ is an
# annotated tag, then return the tag's target instead. # annotated tag, then return the tag's target instead.
def rev_parse_target(revspec) def rev_parse_target(revspec)
...@@ -2409,6 +2405,10 @@ module Gitlab ...@@ -2409,6 +2405,10 @@ module Gitlab
def rev_list_param(spec) def rev_list_param(spec)
spec == :all ? ['--all'] : spec spec == :all ? ['--all'] : spec
end end
def sha_from_ref(ref)
rev_parse_target(ref).oid
end
end end
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment