Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from Sep 15, 2023

Conversation

mj
Copy link
Contributor

@mj mj commented Sep 13, 2023

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

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 fixes voxpupuli#198.
Copy link
Member

@smortex smortex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Not sure about how to label this. For the sake of caution I am tempted to mark this backwards-incompatible. Any thoughts?

@kenyon
Copy link
Member

kenyon commented Sep 15, 2023

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.

@kenyon kenyon merged commit 060279c into voxpupuli:master Sep 15, 2023
32 checks passed
@bk-lg
Copy link

bk-lg commented Sep 16, 2023

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

@smortex
Copy link
Member

smortex commented Sep 16, 2023

@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 😄
Check out https://voxpupuli.org/docs/releasing_version/ if you want to trigger a release.

@mj
Copy link
Contributor Author

mj commented Sep 18, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Installs from influxdata repo can use stable repos
4 participants