move Prometheus retention to startup

wmo-im · Aug 23, 2023 · 06a4881 · 06a4881
1 parent 9e039d2
commit 06a4881
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/docker-compose.monitoring.yml b/docker-compose.monitoring.yml
@@ -58,6 +58,7 @@ services:
       - prometheus-data:/prometheus
     command:
       - '--config.file=/etc/prometheus/prometheus.yml'
+      - '--storage.tsdb.retention.time=10d'
   # cadvisor to monitor our containers!
   cadvisor:
     image: gcr.io/cadvisor/cadvisor:v0.45.0

diff --git a/docs/source/reference/monitoring/index.rst b/docs/source/reference/monitoring/index.rst
@@ -25,8 +25,7 @@ wis2box also analyzes prometheus metrics from MinIO.
 
    For more information see the `list of supported MinIO metrics <https://github.com/minio/minio/blob/master/docs/metrics/prometheus/list.md>`_
 
-The default retention period for Prometheus metrics is 10 days. 
-This value can be modified in the Prometheus configuration at ``prometheus/prometheus.yml``. 
+The default retention period for Prometheus metrics is 10 days.  This value can be modified in the Prometheus startup flag ``--storage.tsdb.retention.time`` in ``docker-compose.monitoring.yml``.
 
 Loki logging
 ------------

diff --git a/prometheus/prometheus.yml b/prometheus/prometheus.yml
@@ -11,11 +11,6 @@ alerting:
     - targets:
       # - alertmanager:9093
 
-storage:
-  tsdb:
-    retention:
-      time: 240h
-
 scrape_configs:
 - job_name: minio-job
   metrics_path: /minio/v2/metrics/cluster