Move Gitaly RefExists check to OPT_OUT

Part of the Gitaly migration as tracked in: https://gitlab.com/gitlab-org/gitaly/issues/401 Acceptance testing was done in: https://gitlab.com/gitlab-org/gitaly/issues/1014

Move Gitaly RefExists check to OPT_OUT
5173bc8a · Zeger-Jan van de Weg · b9d868e3 · 5173bc8a · 5173bc8a
Unverified Commit 5173bc8a authored Apr 12, 2018 by Zeger-Jan van de Weg
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

zj-ref-exists-opt-out.yml changelogs/unreleased/zj-ref-exists-opt-out.yml +5 -0

repository.rb lib/gitlab/git/repository.rb +2 -1

No files found.
--- a/changelogs/unreleased/zj-ref-exists-opt-out.yml
+++ b/changelogs/unreleased/zj-ref-exists-opt-out.yml
+---
+title: Check if a ref exists is done by Gitaly by default
+merge_request:
+author:
+type: performance
--- a/lib/gitlab/git/repository.rb
+++ b/lib/gitlab/git/repository.rb
@@ -297,7 +297,8 @@ module Gitlab
      #
      # Ref names must start with `refs/`.
      def ref_exists?(ref_name)
-        gitaly_migrate(:ref_exists) do |is_enabled|
+        gitaly_migrate(:ref_exists,
+                      status: Gitlab::GitalyClient::MigrationStatus::OPT_OUT) do |is_enabled|
          if is_enabled
            gitaly_ref_exists?(ref_name)
          else