Move most of MetricsController logic to MetricsService

1 files changed, 6 insertions, 35 deletions

diff --git a/app/controllers/metrics_controller.rb b/app/controllers/metrics_controller.rb
index 8b99de06d85..7191a66fe46 100644
--- a/app/controllers/metrics_controller.rb
+++ b/app/controllers/metrics_controller.rb
@@ -1,50 +1,21 @@
-require 'prometheus/client/formats/text'
-
 class MetricsController < ActionController::Base
   protect_from_forgery with: :exception
+  before_action :validate_prometheus_metrics
   include RequiresHealthToken
-  before_action :ensure_prometheus_metrics_are_enabled
-
-  CHECKS = [
-    Gitlab::HealthChecks::DbCheck,
-    Gitlab::HealthChecks::RedisCheck,
-    Gitlab::HealthChecks::FsShardsCheck
-  ].freeze
-
 
   def metrics
-    metrics_text = Prometheus::Client::Formats::Text.marshal_multiprocess(multiprocess_metrics_path)
-    response = "#{health_metrics_text}\n#{metrics_text}"
+    response = "#{metrics_service.health_metrics_text}\n#{metrics_service.prometheus_metrics_text}"
 
     render text: response, content_type: 'text/plain; version=0.0.4'
   end
 
   private
 
-  def ensure_prometheus_metrics_are_enabled
-    return render_404 unless Gitlab::Metrics.prometheus_metrics_enabled?
-  end
-
-  def multiprocess_metrics_path
-    Rails.root.join(ENV['prometheus_multiproc_dir'])
+  def metrics_service
+    @metrics_service ||= MetricsService.new
   end
 
-  def health_metrics_text
-    results = CHECKS.flat_map(&:metrics)
-
-    types = results.map(&:name).uniq.map { |metric_name| "# TYPE #{metric_name} gauge" }
-    metrics = results.map(&method(:metric_to_prom_line))
-
-    types.concat(metrics).join("\n")
-  end
-
-  def metric_to_prom_line(metric)
-    labels = metric.labels&.map { |key, value| "#{key}=\"#{value}\"" }&.join(',') || ''
-
-    if labels.empty?
-      "#{metric.name} #{metric.value}"
-    else
-      "#{metric.name}{#{labels}} #{metric.value}"
-    end
+  def validate_prometheus_metrics
+    render_404 unless Gitlab::Metrics.prometheus_metrics_enabled?
   end
 end