summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLin Jen-Shin <godfat@godfat.org>2019-08-23 17:42:40 +0000
committerLin Jen-Shin <godfat@godfat.org>2019-08-23 17:42:40 +0000
commite84922906e51b22f2f335ebaa4390ccac2e4caf8 (patch)
treed8e797d2dc04b62e005fa2276029b73dec5d9a04
parent15ea614765ce88a93c91a3b055a7c4fb96fd1ccb (diff)
parentebbd2aeb5f56bc2cacc5d95d41da370d8c29069d (diff)
downloadgitlab-ce-e84922906e51b22f2f335ebaa4390ccac2e4caf8.tar.gz
Merge branch 'jc-fix-auto-rugged-detection' into 'master'
Handle when server info doesn't have the storage in question See merge request gitlab-org/gitlab-ce!32023
-rw-r--r--lib/gitlab/gitaly_client.rb3
-rw-r--r--spec/lib/gitlab/gitaly_client_spec.rb10
2 files changed, 12 insertions, 1 deletions
diff --git a/lib/gitlab/gitaly_client.rb b/lib/gitlab/gitaly_client.rb
index e6cbfb00f60..201db9fec26 100644
--- a/lib/gitlab/gitaly_client.rb
+++ b/lib/gitlab/gitaly_client.rb
@@ -406,7 +406,8 @@ module Gitlab
def self.filesystem_id(storage)
response = Gitlab::GitalyClient::ServerService.new(storage).info
storage_status = response.storage_statuses.find { |status| status.storage_name == storage }
- storage_status.filesystem_id
+
+ storage_status&.filesystem_id
end
def self.filesystem_id_from_disk(storage)
diff --git a/spec/lib/gitlab/gitaly_client_spec.rb b/spec/lib/gitlab/gitaly_client_spec.rb
index e9fb6c0125c..99d563e03ec 100644
--- a/spec/lib/gitlab/gitaly_client_spec.rb
+++ b/spec/lib/gitlab/gitaly_client_spec.rb
@@ -27,6 +27,16 @@ describe Gitlab::GitalyClient do
end
end
+ describe '.filesystem_id' do
+ it 'returns an empty string when the storage is not found in the response' do
+ response = double("response")
+ allow(response).to receive(:storage_statuses).and_return([])
+ allow_any_instance_of(Gitlab::GitalyClient::ServerService).to receive(:info).and_return(response)
+
+ expect(described_class.filesystem_id('default')).to eq(nil)
+ end
+ end
+
describe '.stub_class' do
it 'returns the gRPC health check stub' do
expect(described_class.stub_class(:health_check)).to eq(::Grpc::Health::V1::Health::Stub)