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

Always install from the stable release class on Debian and Ubuntu #217

Closed mj closed 9 months ago

mj commented 9 months ago

Pull Request (PR) description

InfluxData have added the release class "stable" to their DEB repository recently and consequently have stopped shipping builds specifically for Debian 12 ("Bookworm"):

https://repos.influxdata.com/debian/dists/

According to the docs in the above URL, using the stable release class is the new preferred way to install Telegraf et al.

This Pull Request (PR) fixes the following issues

Fixes #198

kenyon commented 9 months ago

It looks like the packages you get from all of the distro codenames of https://repos.influxdata.com/ are the same, so effectively this just simplifies the code and doesn't change what packages you get. So I'd say it's backwards compatible.

bk-lg commented 9 months ago

@kenyon is there a rough timeline when this change will be in a release for the forge?

smortex commented 9 months ago

@kenyon thanks! So no label is fine, this will go in the "Other merged PR" in the changelog.

@bk-lg releasing is a two step process and the first one can be done by anybody who think the time has come from a release :smile: Check out https://voxpupuli.org/docs/releasing_version/ if you want to trigger a release.

mj commented 9 months ago

@bk-lg , @smortex, I have prepared the next release in PR #218.