search

voxpupuli / puppet-telegraf

A Puppet module for installing and configuring InfluxData's Telegraf
https://forge.puppet.com/puppet/telegraf
GNU General Public License v3.0
24 stars 99 forks source link

chore: update influxdata repo gpg key #197

Closed powersj closed 1 year ago

powersj commented 1 year ago

Updates the InfluxData repo GPG key to the new rotated key. For more details, see: https://www.influxdata.com/blog/linux-package-signing-key-rotation/

fixes: #196 fixes: #167

sri4kanne commented 1 year ago

We are running into same issue, looks like this PR will help fix it up. Can this be merged please?

kenyon commented 1 year ago

We need to get those tests passing before we can merge it.

powersj commented 1 year ago

We need to get those tests passing before we can merge it.

Fixed, all tests are passing now - I missed the RPM key name

slvr32 commented 1 year ago

@powersj I believe the $_baseurl values in the RedHat section also need updating?

(also removing the conditional for Amazon, if there isn't an Amazon-specific URL?)

$_baseurl = "https://repos.influxdata.com/${telegraf::repo_type}/\$basearch/main"

to be consistent with the RPM-based systems details at https://www.influxdata.com/blog/linux-package-signing-key-rotation/

[influxdata]
name = InfluxData Repository - Stable
baseurl = https://repos.influxdata.com/stable/\$basearch/main
enabled = 1
gpgcheck = 1
gpgkey = https://repos.influxdata.com/influxdata-archive_compat.key
powersj commented 1 year ago

@powersj I believe the $_baseurl values in the RedHat section also need updating?

(also removing the conditional for Amazon, if there isn't an Amazon-specific URL?)

I agree this should be done, but it is not required for the GPG key change that occurred today. I opened up #198 to track moving to the stable repos in a future change.