dj-wasabi / ansible-telegraf

Installing and configuring Telegraf via Ansible for RedHat/Debian/Ubuntu/Windows/Suse.
MIT License
134 stars 116 forks source link

Add Yum repo support for Amazon Linux #79

Closed anthonysr closed 5 years ago

anthonysr commented 5 years ago

Add support for Yum repos on Amazon Linux Adds new variable telegraf_yum_baseurl which selects a different baseurl depending on ansible_distribution

Type of change Bugfix Pull Request

Fixes an issue No existing issue tracked, but without this change you would get the following on Amazon Linux:

fatal: [HOSTNAME]: FAILED! => changed=false msg: |- Failure talking to yum: failure: repodata/repomd.xml from influxdb: [Errno 256] No more mirrors to try. https://repos.influxdata.com/rhel/2018.03/x86_64/stable/repodata/repomd.xml: [Errno 14] HTTPS Error 404 - Not Found fatal: [HOSTNAME]: FAILED! => changed=false

anthonysr commented 5 years ago

This is only failing in molecule for Debian & Ubuntu because "E: Version '1.9.1-1' for 'telegraf' was not found" which is unrelated to this PR.

dj-wasabi commented 5 years ago

Hi @anthonysr-mc

I just merged a PR that it updates the telegraf version to 1.9.2. So I'm approving this PR. In the mean time I think it is best that Molecule will install the latest to prevent the errors you have now.

dj-wasabi commented 5 years ago

And thanks! 👍

(I almost forgot 😄 )