Merge branch 'zj-backup-timeout' into 'master'

Remove timeouts for long running requests

See merge request gitlab-org/gitlab-ce!20810

changelogs/unreleased/zj-backup-timeout.yml

+---
+title: Disable Gitaly timeouts when creating or restoring backups
+merge_request: 20810
+author:
+type: fixed

lib/gitlab/gitaly_client.rb

@@ -407,7 +407,7 @@ module Gitlab
 
     # The default timeout on all Gitaly calls
     def self.default_timeout
-      return 0 if Sidekiq.server?
+      return no_timeout if Sidekiq.server?
 
       timeout(:gitaly_timeout_default)
     end
@@ -420,6 +420,10 @@ module Gitlab
       timeout(:gitaly_timeout_medium)
     end
 
+    def self.no_timeout
+      0
+    end
+
     def self.timeout(timeout_name)
       Gitlab::CurrentSettings.current_application_settings[timeout_name]
     end

lib/gitlab/gitaly_client/repository_service.rb

@@ -202,7 +202,7 @@ module Gitlab
           save_path,
           :create_bundle,
           Gitaly::CreateBundleRequest,
-          GitalyClient.default_timeout
+          GitalyClient.no_timeout
         )
       end
 
@@ -220,7 +220,7 @@ module Gitlab
           bundle_path,
           :create_repository_from_bundle,
           Gitaly::CreateRepositoryFromBundleRequest,
-          GitalyClient.default_timeout
+          GitalyClient.no_timeout
         )
       end
 
@@ -245,7 +245,7 @@ module Gitlab
           :repository_service,
           :create_repository_from_snapshot,
           request,
-          timeout: GitalyClient.default_timeout
+          timeout: GitalyClient.no_timeout
         )
       end