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

exec plugin: allow using glob pattern in command list #1127

Closed
wants to merge 1 commit into from

Conversation

peekeri
Copy link

@peekeri peekeri commented Apr 29, 2016

Improvement idea for the exec plugin:

Allow using glob pattern in the command list in configuration. This enables for
example placing all commands in a single directory and using /path/to/dir/*.sh
as one of the commands to run all shell scripts in that directory.

Glob patterns are applied on every run of the commands, so matching commands can
be added without restarting telegraf.

@sparrc
Copy link
Contributor

sparrc commented May 1, 2016

this looks good, but please update the README, SampleConfig, and changelog

Allow using glob pattern in the command list in configuration. This enables for
example placing all commands in a single directory and using /path/to/dir/*.sh
as one of the commands to run all shell scripts in that directory.

Glob patterns are applied on every run of the commands, so matching commands can
be added without restarting telegraf.
@peekeri
Copy link
Author

peekeri commented May 2, 2016

@sparrc: thanks for the comments. I updated all of those with a remark that it is now possible to use glob patterns in commands.

@sparrc sparrc closed this in 6381efa May 2, 2016
@sparrc
Copy link
Contributor

sparrc commented May 2, 2016

thanks @peekeri

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