Skip to content

Add doc on supported platforms#9100

Merged
reimda merged 5 commits intomasterfrom
supported-platforms
Apr 19, 2021
Merged

Add doc on supported platforms#9100
reimda merged 5 commits intomasterfrom
supported-platforms

Conversation

@reimda
Copy link
Copy Markdown
Contributor

@reimda reimda commented Apr 7, 2021

No description provided.

Copy link
Copy Markdown
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

🤝 ✅ CLA has been signed. Thank you!

@telegraf-tiger telegraf-tiger bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Apr 7, 2021
Copy link
Copy Markdown
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

Comment on lines +5 to +11
Linux
-----
Telegraf intent: *Support latest versions of the most popular distributions*

Telegraf supports RHEL, Fedora, Debian, and Ubuntu. InfluxData provides package repositories for these distributions. Instructions for using the package repositories can be found on [docs.influxdata.com](https://docs.influxdata.com/telegraf/v1.16/introduction/installation/)

https://distrowatch.com/dwres.php?resource=major
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we add a paragraph mentioning that telegraf might also work for older versions and other distributions? I mean I almost never use the distribution version of telegraf but download the static binary from InfluxData directly instead.

Copy link
Copy Markdown
Member

@srebhan srebhan left a comment

Choose a reason for hiding this comment

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

Very nicely written. I added a comment regarding mentioning that other distributions etc. might still work but are not directly supported by InfluxData. It's not a must, but as a user of an alternative OS or distribution you might get the impression that telegraf will not work there, while it does.

@srebhan srebhan self-assigned this Apr 8, 2021
…stributions but is unsupported. Add a link to Telegraf's official docker images
@reimda
Copy link
Copy Markdown
Contributor Author

reimda commented Apr 12, 2021

Very nicely written. I added a comment regarding mentioning that other distributions etc. might still work but are not directly supported by InfluxData. It's not a must, but as a user of an alternative OS or distribution you might get the impression that telegraf will not work there, while it does.

I updated the doc to clarify that other operating systems and linux distributions may work but are not supported.

@reimda reimda requested review from jessingrass and sjwang90 April 12, 2021 23:12
Copy link
Copy Markdown
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

@sjwang90 sjwang90 left a comment

Choose a reason for hiding this comment

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

Super useful!

General comment is describing what the links are. Could even just describe and hyperlink. Just wasn't really sure what the purpose of the link I was clicking on was as I was going through.

Copy link
Copy Markdown
Member

@srebhan srebhan left a comment

Choose a reason for hiding this comment

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

Fine with me but @sjwang90 has some more comments.

@sjwang90 sjwang90 added the docs Issues related to Telegraf documentation and configuration descriptions label Apr 19, 2021
Copy link
Copy Markdown
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

@reimda reimda merged commit 916ac99 into master Apr 19, 2021
@reimda reimda deleted the supported-platforms branch April 19, 2021 22:21
@SomKen
Copy link
Copy Markdown
Contributor

SomKen commented Apr 20, 2021

Awesome for writing this! Does this mean that Influxdb will be building FreeBSD arm packages now?

arstercz pushed a commit to arstercz/telegraf that referenced this pull request Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Issues related to Telegraf documentation and configuration descriptions feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants