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>
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>
This is similar to prometheus_skip_install variable and does what the
name says: when set, it won't try to re-download the node exporter
binary
Signed-off-by: Christian Brabandt <cb@256bit.org>
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>
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>
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>
Move role repo discovery to a script to avoid shell escaping issues.
* Fix up argument_specs.yml.
Signed-off-by: prombot <prometheus-team@googlegroups.com>
Add a script and workflow to automatically check for upstream releases
and create PRs to update the associated role.
Signed-off-by: SuperQ <superq@gmail.com>
Allow long lines in the arguments_spec to avoid line wrapping. This
allows the files to be processed with `yq` without constant diffs and
errors.
Signed-off-by: prombot <prometheus-team@googlegroups.com>
In the docs, the value "latest" was written as a normal string.
By quoting it via `, it marks it explicit as a possible value to accept for node_exporter_version.
Otherwise, it is easy overlooked as acceptable value.
Signed-off-by: Andy Grunwald <andygrunwald@gmail.com>
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>
