search

ansible-community / ansible-nomad

:watch: Ansible role for Nomad
https://galaxy.ansible.com/brianshumate/nomad
BSD 2-Clause "Simplified" License
298 stars 166 forks source link

Add telemetry configuration possibilites #88

Closed rndmh3ro closed 4 years ago

ducminhle commented 4 years ago

Hello @rndmh3ro, Do we need to add telemetry config? I see we have nomad_config_custom, I think we can add telemetry config in this variable. Ex:

nomad_config_custom: telemetry: publish_allocation_metrics: true publish_node_metrics: true prometheus_metrics: true

rndmh3ro commented 4 years ago

Hey @ducminhle,

we probably don't need this. However this way it's documented and easier to use.

rndmh3ro commented 4 years ago

Any reasons against merging this?