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

New option to filter the plugins to run at startup #57

Closed
wants to merge 2 commits into from
Closed

New option to filter the plugins to run at startup #57

wants to merge 2 commits into from

Conversation

zepouet
Copy link
Contributor

@zepouet zepouet commented Jul 9, 2015

Hello,

I just add a new option to filter plugins to enable at startup.

In the future, we will have a lot of plugins into all.go and into config.toml.
A solution would be to have many differents config files.
Another one could be to use this option to filter the plugins.

go run cmd/telegraf/telegraf.go --config telegraf.toml -filter mysql:redis

I hope it is a good idea. For me it is nice to understand the code behind and now how to write plugin.
Best regards,

Nicolas

@zepouet zepouet changed the title New option to filter the plugin to run at startup New option to filter the plugins to run at startup Jul 10, 2015
@evanphx
Copy link
Contributor

evanphx commented Jul 21, 2015

What's the reason for being able to filter the plugins like this?

@zepouet
Copy link
Contributor Author

zepouet commented Jul 21, 2015

Hello,

Maybe I am wrong but If I want to use the same config file and activate only one plugin, the filter option could be nice. Else I must duplicate config file and delete other plugin to disable them.
The config file share common configuration so it is nice to avoid to duplicate it.

Thanks for reading this PR.

Best regards,
Nicolas

@sparrc
Copy link
Contributor

sparrc commented Aug 5, 2015

@zepouet Please update the README with an example under Quickstart -> How to use it and resolve the branch conflicts so I can merge it, thanks!

@zepouet
Copy link
Contributor Author

zepouet commented Aug 11, 2015

Hello,

I close this PR because I created a new one with more comment and file for running unit test. : #106

Sorry I deleted my local branch by mistake (my fork) so I could not merge as you ask me :(
Thanks a lot.

Best regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants