Commit graph

12 commits

Author SHA1 Message Date
Kevin Bowrin
be0a877b00
node_exporter: Fix Systemd ProtectHome option in service unit
Fixes an issue with the jinja2 snippet which is used to create the node_exporter Systemd unit. More details here: https://github.com/prometheus-community/ansible/issues/13

Jinja2 namespaces are used to ensure the variable `protect_home` can be set in the parent scope of the `for` loop looking through the mounts.

Signed-off-by: Kevin Bowrin <kevinbowrin@cunet.carleton.ca>
2023-05-09 16:13:50 -04:00
Ilari Iso-Junno
9812360144 enhancement: support ansible-vaulted basic auth passwords
If ansible-vaulted password is passed to password_hash filter, it will throw and error:
'secret must be unicode or bytes, not ansible.parsing.yaml.objects.AnsibleVaultEncryptedUnicode'
Handling the vaulted password with string filter first will avoid the problem.

Signed-off-by: Ilari Iso-Junno <iisojunn@users.noreply.github.com>
2023-04-12 10:45:46 +03:00
Andrew Kingston
7437ac0a64 fix: correct quotation of flags in systemd config file
Correct the quotation around the collector option flags and values
in the node_exporter role systemd unit config file template so that
regex values get parsed correctly.

Refs: #68
Signed-off-by: Andrew Kingston <andrew.kingston@framestore.com>
2023-03-20 12:27:42 +00:00
Andrew Kingston
a2fa58b772 fix: remove quotation of flags in systemd config file
Remove the quotation around the collector option flags and values
in the node_exporter role systemd unit config file template so that
regex values get parsed correctly.

Refs: #68
Signed-off-by: Andrew Kingston <andrew.kingston@framestore.com>
2023-03-20 12:27:42 +00:00
Andrew Kingston
ebae6c7610 fix: correct quotation of flags in systemd config file
Correct the quotation around the collector option flags and values
in the node_exporter role systemd unit config file template so that
regex values get parsed correctly.

Refs: #68
Signed-off-by: Andrew Kingston <andrew.kingston@framestore.com>
2023-03-20 12:27:42 +00:00
Ben Kochie
4f027625b3
Merge pull request #65 from gardar/fix-var-style
refactor: underscore variable prefix should be reserved...
2023-03-13 13:39:26 +01:00
gardar
585ac79c46
refactor: underscore prefix should be reserved for nonconfigurable internal variables
Signed-off-by: gardar <gardar@users.noreply.github.com>
2023-03-09 17:56:31 +00:00
gardar
e02e6a5976
refactor: avoid using command module for systemd version fact
Signed-off-by: gardar <gardar@users.noreply.github.com>
2023-03-08 18:03:00 +00:00
Ben Kochie
bed08e8a45
Merge pull request #12 from cudevmaxwell/issue-11
node_exporter: Fix broken config file CLI flag name
2022-12-24 18:47:28 +01:00
Kevin Bowrin
5d47937e3b
node_exporter: Support filesystems mounted under '/home'
Set systemd's ProtectHome to 'read-only' instead of
'yes' if there exists filesystems which are mounted
at or under '/home', instead of at '/home' exclusively.

Fixes #13.

Signed-off-by: Kevin Bowrin <kevinbowrin@cunet.carleton.ca>
2022-12-23 16:08:26 -05:00
Kevin Bowrin
3fb29ee4b8
Fix broken config file CLI flag name
Fixes #11.

Uses a version test to ensure backwards compatibility.

Signed-off-by: Kevin Bowrin <kevinbowrin@cunet.carleton.ca>
2022-12-23 14:15:07 -05:00
Paweł Krupa (paulfantom)
dcfbfa84d2 initial migration of roles from cloudalchemy
Signed-off-by: Paweł Krupa (paulfantom) <pawel@krupa.net.pl>
2022-09-23 13:29:55 +02:00