From 313ecd8f8d4eb5999ca2623838e70d270eff77b0 Mon Sep 17 00:00:00 2001 From: Slavi Pantaleev Date: Sat, 13 Jan 2024 15:56:49 +0200 Subject: [PATCH] Do not require Prometheus in matrix-prometheus-nginxlog-exporter The user may be running Prometheus elsewhere. It doesn't need to be getting installed using the current playbook. --- .../tasks/validate_config.yml | 7 ------- 1 file changed, 7 deletions(-) diff --git a/roles/custom/matrix-prometheus-nginxlog-exporter/tasks/validate_config.yml b/roles/custom/matrix-prometheus-nginxlog-exporter/tasks/validate_config.yml index 009af0b9c..8519c72e3 100644 --- a/roles/custom/matrix-prometheus-nginxlog-exporter/tasks/validate_config.yml +++ b/roles/custom/matrix-prometheus-nginxlog-exporter/tasks/validate_config.yml @@ -1,12 +1,5 @@ --- -- name: Fail if Prometheus not enabled - ansible.builtin.fail: - msg: > - You need to set `prometheus_enabled: true` or configure an external Prometheus database - as this roles requires Prometheus for data storage. - when: not prometheus_enabled and not matrix_prometheus_nginxlog_exporter_metrics_proxying_enabled - - name: Fail if docker image not availble for arch ansible.builtin.fail: msg: >