-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Merging a number of plugins #5781
Comments
Woah, that sounds like a lot of great work! There is active work this week on enhancing the current ping plugin to do essentially what you've described. It'd be great to check out what you've got for that and other plugins. I think what would work well would be to open a pull request for each plugin that we can individually review and merge in as appropriate. I'm sure we can help prioritize them before you spend time opening all those PR's. |
@srclosson glad to hear from you! I'd love to see these plugins added! |
As mentioned, we do need to evaluate these one at a time, and it will take some time to go through the full list. I suggest picking the top one or two you want first and open pull requests with the code. Here are some initial reactions to help you pick which ones might be good candiates to get started. One thing to keep in mind is even if we don't accept a plugin, there is a future path where they could live outside of Telegraf in a way that wouldn't require you to recompile them for every Telegraf release. There isn't a timeline for this, and honestly we don't know how exactly it will happen (probably grpc), but it is something we want to provide. Inputs:
Outputs:
Processors:
Service Plugins: 1 & 2. It's really cool how you were able to use Telegraf successfully for these uses but I think we probably wouldn't add these since they deal with the query side of things. |
File directory monitoring plugin (dirmon input + transfer output) merged in #8751. Closing issue. @srclosson If you would like to contribute any of your other plugins, we'd love any PRs 😄 |
Feature Request
Hi Team, over the years, I've developed a number of plugins, and would like to discuss how I can get all of these "up to snuff" and merged into the main code. I expect there will be a little work just to get them all documented properly, but I feel at least submitting a report and starting off with some discussion of order would be helpful.
Proposal:
The current new plugins I've developed are:
Input Plugins
Output Plugins
Processor Plugins
Service Plugins
A service plugin was a new plugin type that I added. It's essentially a stand alone service. Hopefully the plugins for it make sense, as I felt they didn't fit into any other category. A service plugin has a built-in pub-sub messaging bus that enables service plugins to send messages to other service plugins, but the message format is unstructured, and I've not used it yet.
Use case:
I would like to have some place to tackle all these plugins in a somewhat orderly way.
I've written several serializer and parser plugins, which I will update this issue with as well.
The text was updated successfully, but these errors were encountered: