search

mesaguy / ansible-prometheus

Ansible role for the management of Prometheus software and Prometheus exporters
https://galaxy.ansible.com/mesaguy/prometheus
MIT License
71 stars 31 forks source link

nginx_exporter_nginxinc is broken because of kebab-case variable names #32

Closed spy-t closed 3 years ago

spy-t commented 3 years ago

The nginx_exporter_nginxinc exporter is broken because it declares it's software name as nginx-prometheus-exporter_nginxinc. When overriding the default variables the resulting variable names (eg prometheus_nginx-prometheus-exporter_nginxinc_host) can not be processed by ansible resulting in 

ERROR! Invalid variable name in vars specified for IncludeRole: 'prometheus_nginx-prometheus-exporter_nginxinc_host' is not a valid variable name