Skip to content

fix!: Remove deprecated plugins#9609

Closed
reimda wants to merge 2 commits intomasterfrom
deprecated-plugins
Closed

fix!: Remove deprecated plugins#9609
reimda wants to merge 2 commits intomasterfrom
deprecated-plugins

Conversation

@reimda
Copy link
Copy Markdown
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
Copy Markdown
Contributor

Hipska commented Sep 6, 2022

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

@reimda
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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