search

mrlesmithjr / ansible-telegraf

MIT License
0 stars 4 forks source link

Feature: Add support for input plugin tags #1

Open adilnaimi opened 7 years ago

adilnaimi commented 7 years ago

This is not really an issue, it just would be nice to be able to add support for the input plugins tags.

Telegraf saying that the order matters, the [inputs.cpu.tags] table must be at the end of the plugin definition. Example:

[[inputs.cpu]]
  percpu = false
  totalcpu = true
  [inputs.cpu.tags]
    tag1 = "foo"
    tag2 = "bar"
mrlesmithjr commented 7 years ago

@adilnaimi Thanks for raising this. I have added the initial functionality for CPU/Disk tagging. Would you mind testing this out? issue-#1 is the branch to test against if you are up to it.

adilnaimi commented 7 years ago

yes it works thank you.

A minor correction: in main.yml the - include: config.yml is looking for config_telegraf, while the default variable is telagraf_config

mrlesmithjr commented 7 years ago

Great and no problem.

Thanks for pointing that out. Will fix this and push up to master.