diff options
| author | Stan Hu <stanhu@gmail.com> | 2019-07-19 10:01:09 -0700 |
|---|---|---|
| committer | Stan Hu <stanhu@gmail.com> | 2019-07-19 10:03:41 -0700 |
| commit | ec5ceae623fceff0a959c7d297970d37285532dc (patch) | |
| tree | 1cf6083ee561298c1c1f5c652ecf315f95238d9a /lib/gitlab/gitaly_client.rb | |
| parent | 2e56b134687ccc8043f14eef4c5d0bfa9e1fd370 (diff) | |
| download | gitlab-ce-ec5ceae623fceff0a959c7d297970d37285532dc.tar.gz | |
Fix Gitaly auto-detection caching
If `GitalyClient#can_use_disk?` returned `false`, it was never cached
properly and led to excessive number of Gitaly calls. Instead of using
`cached_value.present?`, we need to check `cached_value.nil?`.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/64802
Diffstat (limited to 'lib/gitlab/gitaly_client.rb')
| -rw-r--r-- | lib/gitlab/gitaly_client.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/gitlab/gitaly_client.rb b/lib/gitlab/gitaly_client.rb index e7319a7b7f0..c98de722fe1 100644 --- a/lib/gitlab/gitaly_client.rb +++ b/lib/gitlab/gitaly_client.rb @@ -392,7 +392,7 @@ module Gitlab @can_use_disk[storage] end - return cached_value if cached_value.present? + return cached_value unless cached_value.nil? gitaly_filesystem_id = filesystem_id(storage) direct_filesystem_id = filesystem_id_from_disk(storage) |