Paweł Krupa

Both values come from `list_health()` in stats API. The left one is `tx_bytes-r` from LAN subsystem, the right one is `tx_bytes-r` from WLAN subsystem.

Yeah, that's from https://api-browser-demo.artofwifi.net/# I took a look at JS code from unifi demo site at https://demo.ui.com/manage/site/default/dashboard. The object responsible for left arrow is in HTML div with class `unifiNetworkStatusDiagramConnectorSpeed--withLeftArrowAfter`,...

What is the ansible output from running the role? Could you run it with higher verbosity and attach output of `Copy the node_exporter config file` task?

Could you add this package requirement to https://github.com/cloudalchemy/ansible-node-exporter#requirements section in README? It seems like an ansible bug in `password_hash` jinja function. I would expect it to fail if bcrypt is...

Seems like a fix would be to always quote with double quotes and not single ones. Unquoted regex can also lead to unexpected issues as mentioned in https://github.com/cloudalchemy/ansible-node-exporter/pull/144

Whole idea behind having one variable per flag is to have a mechanism which quickly prevents misconfiguration of node_exporter before ansible even gets to install stage. This is covered in...

> I'll note that node_exporter refuses to start with incorrect/unknown flags, so there is no risk of a mistake going unnoticed. True, but the current mechanism is more about upgrades...

> In that case having separate variables for each flag is the only way to go. Consider this enhancement request officially changed :) PRs for new variables are welcome :)

@ritwiksin : Since this is a collector, you can use similar syntax to textfile collector example in https://github.com/cloudalchemy/ansible-node-exporter/blob/0460c128f8d861d6aa28df83fb15226dfec54269/defaults/main.yml#L14-L17 In your case it will be: ``` node_exporter_enabled_collectors: - supervisord: url: ```

> Adding support for the new node_exporter flags is probably what we want. > In that case having separate variables for each flag is the only way to go. Consider...