diff options
author | Aleksei Lipniagov <alipniagov@gitlab.com> | 2019-09-11 19:50:38 +0300 |
---|---|---|
committer | Aleksei Lipniagov <alipniagov@gitlab.com> | 2019-09-11 19:59:18 +0300 |
commit | 21eb312530448e76467d8e36131453636a4104a1 (patch) | |
tree | cea41006858975f64086e57dad0a4b1a2760cf6a | |
parent | 3c282a88aa25129c15d219351852e1ea1f2b22b6 (diff) | |
download | gitlab-ce-66889-investigate-missing-metric.tar.gz |
Log metric dir cleanup, keep gauge for master66889-investigate-missing-metric
Do not remove particular metric to investigate:
https://gitlab.com/gitlab-org/gitlab-ce/issues/66889
-rw-r--r-- | lib/prometheus/cleanup_multiproc_dir_service.rb | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/lib/prometheus/cleanup_multiproc_dir_service.rb b/lib/prometheus/cleanup_multiproc_dir_service.rb index 6418b4de166..acce6d5f8f5 100644 --- a/lib/prometheus/cleanup_multiproc_dir_service.rb +++ b/lib/prometheus/cleanup_multiproc_dir_service.rb @@ -6,18 +6,29 @@ module Prometheus def execute FileUtils.rm_rf(old_metrics) if old_metrics + + log_cleanup end private def old_metrics strong_memoize(:old_metrics) do - Dir[File.join(multiprocess_files_dir, '*.db')] if multiprocess_files_dir + if multiprocess_files_dir + # TODO: remove `.select {}` part after investigating + # https://gitlab.com/gitlab-org/gitlab-ce/issues/66889 + Dir[File.join(multiprocess_files_dir, '*.db')].select {|f| f !~ /gauge_all_unicorn_master/} + end end end def multiprocess_files_dir ::Prometheus::Client.configuration.multiprocess_files_dir end + + def log_cleanup + message = "Cleanup Prometheus multiprocess_files_dir, pid: #{Process.pid}" + Gitlab::AppLogger.info(message) + end end end |