From 1bc3805dedf3c547a67fa920bdfe3a7dccdd3e58 Mon Sep 17 00:00:00 2001
From: Timo Makinen <tmakinen@foo.sh>
Date: Fri, 10 Jan 2025 07:30:32 +0000
Subject: [PATCH] node_exporter: Fix startup options for Fedora

---
 roles/node_exporter/tasks/main.yml | 20 ++++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/roles/node_exporter/tasks/main.yml b/roles/node_exporter/tasks/main.yml
index a873906..afb5e76 100644
--- a/roles/node_exporter/tasks/main.yml
+++ b/roles/node_exporter/tasks/main.yml
@@ -69,7 +69,7 @@
     job: /usr/local/sbin/node-exporter-run-textfile-collector
     minute: "*/10"
 
-- name: Modify config
+- name: Modify config (pre 1.5.0)
   ansible.builtin.lineinfile:
     path: /etc/default/prometheus-node-exporter
     regexp: "^ARGS="
@@ -80,7 +80,23 @@
       --web.config=/etc/node_exporter/web-config.yml
       --collector.textfile.directory=/var/lib/prometheus/node-exporter"
   notify: Restart node_exporter
-  when: ansible_os_family == "RedHat"
+  when:
+    - ansible_os_family == "RedHat"
+    - ansible_distribution != "Fedora"
+
+- name: Modify config
+  ansible.builtin.lineinfile:
+    path: /etc/default/prometheus-node-exporter
+    regexp: "^ARGS="
+    line: >-
+      ARGS="--collector.filesystem.ignored-mount-points='^/(dev|proc|sys|run/(user|credentials/systemd-.+))($|/)'
+      --collector.netclass.ignored-devices='^(br-|docker|veth).+$'
+      --collector.netdev.device-exclude='^(br-|docker|veth).+$'
+      --web.config.file=/etc/node_exporter/web-config.yml
+      --collector.textfile.directory=/var/lib/prometheus/node-exporter"
+  notify: Restart node_exporter
+  when:
+    - ansible_distribution == "Fedora"
 
 - name: Install disk and raid monitoring scripts
   ansible.builtin.copy: