Simplify Redis Health checks

Intriduced RedisAbstractCheck that implements
metric_prefis and redis_instance_class_name based on class name

lib/gitlab/health_checks/redis/cache_check.rb

@@ -4,31 +4,7 @@ module Gitlab
   module HealthChecks
     module Redis
       class CacheCheck
-        extend SimpleAbstractCheck
-
-        class << self
-          def check_up
-            check
-          end
-
-          private
-
-          def metric_prefix
-            'redis_cache_ping'
-          end
-
-          def successful?(result)
-            result == 'PONG'
-          end
-
-          # rubocop: disable CodeReuse/ActiveRecord
-          def check
-            catch_timeout 10.seconds do
-              Gitlab::Redis::Cache.with(&:ping)
-            end
-          end
-          # rubocop: enable CodeReuse/ActiveRecord
-        end
+        extend RedisAbstractCheck
       end
     end
   end

lib/gitlab/health_checks/redis/queues_check.rb

@@ -4,31 +4,7 @@ module Gitlab
   module HealthChecks
     module Redis
       class QueuesCheck
-        extend SimpleAbstractCheck
-
-        class << self
-          def check_up
-            check
-          end
-
-          private
-
-          def metric_prefix
-            'redis_queues_ping'
-          end
-
-          def successful?(result)
-            result == 'PONG'
-          end
-
-          # rubocop: disable CodeReuse/ActiveRecord
-          def check
-            catch_timeout 10.seconds do
-              Gitlab::Redis::Queues.with(&:ping)
-            end
-          end
-          # rubocop: enable CodeReuse/ActiveRecord
-        end
+        extend RedisAbstractCheck
       end
     end
   end

lib/gitlab/health_checks/redis/rate_limiting_check.rb

@@ -4,31 +4,7 @@ module Gitlab
   module HealthChecks
     module Redis
       class RateLimitingCheck
-        extend SimpleAbstractCheck
-
-        class << self
-          def check_up
-            check
-          end
-
-          private
-
-          def metric_prefix
-            'redis_rate_limiting_ping'
-          end
-
-          def successful?(result)
-            result == 'PONG'
-          end
-
-          # rubocop: disable CodeReuse/ActiveRecord
-          def check
-            catch_timeout 10.seconds do
-              Gitlab::Redis::RateLimiting.with(&:ping)
-            end
-          end
-          # rubocop: enable CodeReuse/ActiveRecord
-        end
+        extend RedisAbstractCheck
       end
     end
   end

lib/gitlab/health_checks/redis/redis_abstract_check.rb

+# frozen_string_literal: true
+
+module Gitlab
+  module HealthChecks
+    module Redis
+      module RedisAbstractCheck
+        include SimpleAbstractCheck
+
+        def check_up
+          successful?(check)
+        end
+
+        private
+
+        def redis_instance_class_name
+          Gitlab::Redis.const_get(redis_instance_name.camelize , false)
+        end
+
+        def metric_prefix
+          "redis_#{redis_instance_name}_ping"
+        end
+
+        def redis_instance_name
+          name.sub(/_check$/, '')
+        end
+
+        def successful?(result)
+          result == 'PONG'
+        end
+
+        # rubocop: disable CodeReuse/ActiveRecord
+        def check
+          catch_timeout 10.seconds do
+            redis_instance_class_name.with(&:ping)
+          end
+        end
+        # rubocop: enable CodeReuse/ActiveRecord
+      end
+    end
+  end
+end

lib/gitlab/health_checks/redis/redis_check.rb

@@ -14,16 +14,22 @@ module Gitlab
           end
 
           def successful?(result)
-            result == 'PONG'
+            result == true
           end
 
           def check
-            ::Gitlab::HealthChecks::Redis::CacheCheck.check_up &&
-              ::Gitlab::HealthChecks::Redis::QueuesCheck.check_up &&
-              ::Gitlab::HealthChecks::Redis::SharedStateCheck.check_up &&
-              ::Gitlab::HealthChecks::Redis::TraceChunksCheck.check_up &&
-              ::Gitlab::HealthChecks::Redis::RateLimitingCheck.check_up &&
-              ::Gitlab::HealthChecks::Redis::SessionsCheck.check_up
+            redis_health_checks.all?(&:check_up)
+          end
+
+          def redis_health_checks
+            [
+              Gitlab::HealthChecks::Redis::CacheCheck,
+              Gitlab::HealthChecks::Redis::QueuesCheck,
+              Gitlab::HealthChecks::Redis::SharedStateCheck,
+              Gitlab::HealthChecks::Redis::TraceChunksCheck,
+              Gitlab::HealthChecks::Redis::RateLimitingCheck,
+              Gitlab::HealthChecks::Redis::SessionsCheck
+            ]
           end
         end
       end

lib/gitlab/health_checks/redis/sessions_check.rb

@@ -4,31 +4,7 @@ module Gitlab
   module HealthChecks
     module Redis
       class SessionsCheck
-        extend SimpleAbstractCheck
-
-        class << self
-          def check_up
-            check
-          end
-
-          private
-
-          def metric_prefix
-            'redis_sessions_ping'
-          end
-
-          def successful?(result)
-            result == 'PONG'
-          end
-
-          # rubocop: disable CodeReuse/ActiveRecord
-          def check
-            catch_timeout 10.seconds do
-              Gitlab::Redis::Sessions.with(&:ping)
-            end
-          end
-          # rubocop: enable CodeReuse/ActiveRecord
-        end
+        extend RedisAbstractCheck
       end
     end
   end

lib/gitlab/health_checks/redis/shared_state_check.rb

@@ -4,31 +4,7 @@ module Gitlab
   module HealthChecks
     module Redis
       class SharedStateCheck
-        extend SimpleAbstractCheck
-
-        class << self
-          def check_up
-            check
-          end
-
-          private
-
-          def metric_prefix
-            'redis_shared_state_ping'
-          end
-
-          def successful?(result)
-            result == 'PONG'
-          end
-
-          # rubocop: disable CodeReuse/ActiveRecord
-          def check
-            catch_timeout 10.seconds do
-              Gitlab::Redis::SharedState.with(&:ping)
-            end
-          end
-          # rubocop: enable CodeReuse/ActiveRecord
-        end
+        extend RedisAbstractCheck
       end
     end
   end

lib/gitlab/health_checks/redis/trace_chunks_check.rb

@@ -4,31 +4,7 @@ module Gitlab
   module HealthChecks
     module Redis
       class TraceChunksCheck
-        extend SimpleAbstractCheck
-
-        class << self
-          def check_up
-            check
-          end
-
-          private
-
-          def metric_prefix
-            'redis_trace_chunks_ping'
-          end
-
-          def successful?(result)
-            result == 'PONG'
-          end
-
-          # rubocop: disable CodeReuse/ActiveRecord
-          def check
-            catch_timeout 10.seconds do
-              Gitlab::Redis::TraceChunks.with(&:ping)
-            end
-          end
-          # rubocop: enable CodeReuse/ActiveRecord
-        end
+        extend RedisAbstractCheck
       end
     end
   end