Merge branch 'zj-ref-exists-opt-out' into 'master'

Move Gitaly RefExists check to OPT_OUT Closes gitaly#401 See merge request gitlab-org/gitlab-ce!18333

Merge branch 'zj-ref-exists-opt-out' into 'master'
b3ed751c · Nick Thomas · 8feab949 · 5173bc8a · b3ed751c · b3ed751c
Commit b3ed751c authored Apr 17, 2018 by Nick Thomas
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
@@ -300,7 +300,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