Compute repository checksum by default

Acceptance testing was done in: https://gitlab.com/gitlab-org/gitaly/issues/1146. Errors were limited to NotFound errors, as the repository really doesn't exists. Closes https://gitlab.com/gitlab-org/gitaly/issues/1146 Optimistically moved https://gitlab.com/gitlab-org/gitaly/issues/1105 to opt out on the Gitaly migration board.

Compute repository checksum by default
27d2f407 · Zeger-Jan van de Weg · 87f1736a · 27d2f407 · 27d2f407
Unverified Commit 27d2f407 authored Apr 30, 2018 by Zeger-Jan van de Weg
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

zj-repo-checksum-opt-out.yml changelogs/unreleased/zj-repo-checksum-opt-out.yml +5 -0

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

No files found.
--- a/changelogs/unreleased/zj-repo-checksum-opt-out.yml
+++ b/changelogs/unreleased/zj-repo-checksum-opt-out.yml
+---
+title: Compute Gitlab::Git::Repository#checksum on Gitaly by default
+merge_request:
+author:
+type: performance
--- a/lib/gitlab/git/repository.rb
+++ b/lib/gitlab/git/repository.rb
@@ -1546,7 +1546,8 @@ module Gitlab
      end

      def checksum
-        gitaly_migrate(:calculate_checksum) do |is_enabled|
+        gitaly_migrate(:calculate_checksum,
+                      status: Gitlab::GitalyClient::MigrationStatus::OPT_OUT) do |is_enabled|
          if is_enabled
            gitaly_repository_client.calculate_checksum
          else