Fachschaft/nixConfig
5
2
Fork 2
Code Issues 7 Pull requests 1 Projects 1 Wiki Activity

Enable prometheus node exporter by default #19

Merged
nerf merged 5 commits from Gonne/nixConfig:prometheusNodeExporter into main 2023-11-06 14:30:33 +00:00
Conversation 5 Commits 5 Files changed 2 +3 -1
2 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 2c2b24d0a9 - Show all commits

2
nixos/roles/default.nix
View file

@ -56,5 +56,5 @@ services = {
PasswordAuthentication = false; PasswordAuthentication = false;
}; };
}; };
}; };
Gonne marked this conversation as resolved Outdated
nerf commented 2023-10-28 14:29:08 +00:00
Outdated
Review
Copy link

This is the closing bracket of the services record, there should be less white space, right?

This is the closing bracket of the services record, there should be less white space, right?
} }

2
nixos/roles/prometheusNodeExporter.nix
View file

@ -1,3 +1,4 @@
{config, ...}:
Gonne marked this conversation as resolved Outdated
nerf commented 2023-10-28 14:25:17 +00:00
Outdated
Review
Copy link

At least there should be persistence for
/var/lib/${config.services.prometheus.stateDir}
(maybe the config name is slightly wrong better look it up.)

At least there should be persistence for `/var/lib/${config.services.prometheus.stateDir}` (maybe the config name is slightly wrong better look it up.)
{ {
imports = [ ]; imports = [ ];
services.prometheus.exporters.node = { services.prometheus.exporters.node = {
@ -33,4 +34,5 @@
]; ];
}; };
networking.firewall.allowedTCPPorts = [ 9100 ]; networking.firewall.allowedTCPPorts = [ 9100 ];
environment.persistence.${config.impermanence.name}.directories = [ "/var/lib/${config.services.prometheus.stateDir}" ];
} }