Remove FDW configured check

Since GitLab 13.2 we don't rely on FDW query/models
parent 6ae03e96
......@@ -12,7 +12,6 @@ module Gitlab
return 'Geo node has a database that is writable which is an indication it is not configured for replication with the primary node.' unless Gitlab::Database.db_read_only?
return 'Geo node does not appear to be replicating the database from the primary node.' if replication_enabled? && !replication_working?
return "Geo database version (#{database_version}) does not match latest migration (#{migration_version}).\nYou may have to run `gitlab-rake geo:db:migrate` as root on the secondary." unless database_migration_version_match?
return 'Geo database is not configured to use Foreign Data Wrapper.' unless Gitlab::Geo::Fdw.enabled?
''
rescue => e
......
......@@ -102,7 +102,6 @@ RSpec.describe Gitlab::Geo::HealthCheck, :geo do
context 'that is working' do
before do
allow(subject).to receive(:replication_working?).and_return(true)
allow(Gitlab::Geo::Fdw).to receive(:enabled?) { true }
end
it 'returns an error if database is not fully migrated' do
......@@ -115,12 +114,6 @@ RSpec.describe Gitlab::Geo::HealthCheck, :geo do
expect(message).to include('gitlab-rake geo:db:migrate')
end
it 'returns an error when FDW is disabled' do
allow(Gitlab::Geo::Fdw).to receive(:enabled?) { false }
expect(subject.perform_checks).to match(/Geo database is not configured to use Foreign Data Wrapper/)
end
it 'finally returns an empty string when everything is healthy' do
expect(subject.perform_checks).to be_blank
end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment