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

fix!: Remove deprecated plugins #9609

Closed
wants to merge 2 commits into from
Closed

fix!: Remove deprecated plugins #9609

wants to merge 2 commits into from

Conversation

reimda
Copy link
Contributor

@reimda reimda commented Aug 10, 2021

Remove the plugins listed in #9478. Since this breaks compatibility, it will not be merged until the next major release (2.0).

Telegraf will also need to tell upgrading users of these plugins to migrate. This PR is a draft because it doesn't handle migration yet.

@telegraf-tiger telegraf-tiger bot added the fix pr to fix corresponding bug label Aug 10, 2021
@Hipska Hipska added this to the 2.0.0 milestone Oct 14, 2021
@sspaink sspaink added the breaking change Improvement to Telegraf that requires changes to the plugin or agent; for minor/major releases label Aug 31, 2022
@Hipska
Copy link
Contributor

Hipska commented Sep 6, 2022

What about the plugins/*/deprecations.go files?

@reimda
Copy link
Contributor Author

reimda commented Sep 6, 2022

What about the plugins/*/deprecations.go files?

This PR was from before we added deprecations.go. It definitely needs an update

@Hipska
Copy link
Contributor

Hipska commented Sep 6, 2022

Oh, I see indeed..

@sspaink sspaink added the 2.0 label Sep 12, 2022
@Hipska Hipska removed the 2.0 label Sep 19, 2022
@powersj powersj removed this from the 2.0.0 milestone Feb 2, 2023
@powersj
Copy link
Contributor

powersj commented Feb 15, 2023

Closing this to re-open without conflicts + add a change log message with breaking change.

@powersj powersj closed this Feb 15, 2023
@powersj powersj deleted the deprecated-plugins branch January 4, 2024 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Improvement to Telegraf that requires changes to the plugin or agent; for minor/major releases fix pr to fix corresponding bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants