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

feat: migrate input plugins to new sample config format (M-Z) #10926

Merged
merged 13 commits into from
Apr 11, 2022

Conversation

sspaink
Copy link
Contributor

@sspaink sspaink commented Mar 31, 2022

Depends on: #10886

This migrates all input plugins (M-Z) to follow the new sample config format introduced in #10886.

This is a separate PR in an attempt to keep #10886 smaller and more easily reviewable.

The CI is expected to fail until #10886 is merged

@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 Mar 31, 2022
@sspaink sspaink self-assigned this Apr 5, 2022
@sspaink sspaink force-pushed the updateinputconifg_m-z branch from 2596287 to 038a7ae Compare April 7, 2022 16:22
@sspaink sspaink force-pushed the updateinputconifg_m-z branch from b0c662a to 0ccd913 Compare April 11, 2022 17:53
@sspaink sspaink marked this pull request as ready for review April 11, 2022 17:53
@sspaink sspaink added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Apr 11, 2022
@telegraf-tiger
Copy link
Contributor

Copy link
Contributor

@powersj powersj left a comment

Choose a reason for hiding this comment

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

Diff is so long it didn't even load all the files for me :) Anyway, thanks for doing all of this, I think I have gone through them all enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin 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.

2 participants