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

docs: update deprecated plugin readmes #10100

Merged
merged 12 commits into from
Nov 16, 2021

Conversation

sjwang90
Copy link
Contributor

Added note for all deprecated plugins - Included version plugin was deprecated and replacement plugin

@telegraf-tiger
Copy link
Contributor

Copy link
Contributor

@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.

Looks good to me.

@srebhan srebhan self-assigned this Nov 15, 2021
@srebhan srebhan added docs Issues related to Telegraf documentation and configuration descriptions ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. labels Nov 15, 2021
@sspaink sspaink merged commit f71695b into influxdata:master Nov 16, 2021
powersj pushed a commit that referenced this pull request Nov 17, 2021
@Hipska
Copy link
Contributor

Hipska commented Nov 18, 2021

Can the inconsistent references to the new plugins be aligned please?

  • ../<new-plugin>
  • /plugins/<type>/<new-plugin>
  • https://github.com/influxdata/telegraf/tree/master/plugins/<type>/<new-plugin>

@sjwang90 sjwang90 mentioned this pull request Nov 19, 2021
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 ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants