Commit 9deaf60d authored by Stan Hu's avatar Stan Hu

Add more descriptive message about NotImplementedError and display it in health check

parent de431bbc
......@@ -6,6 +6,8 @@ class GeoNodeStatus
def health
@health ||= HealthCheck::Utils.process_checks(['geo'])
rescue NotImplementedError => e
@health = e.to_s
end
def healthy?
......
......@@ -2,7 +2,7 @@ module Gitlab
module Geo
class HealthCheck
def self.perform_checks
raise NotImplementedError unless Gitlab::Database.postgresql?
raise NotImplementedError.new('Geo is only compatible with PostgreSQL') unless Gitlab::Database.postgresql?
return '' unless Gitlab::Geo.secondary?
return 'The Geo secondary role is disabled.' unless Gitlab::Geo.secondary_role_enabled?
......
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