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

Convert the telegraf_plugins_extra varaible to a hash so that we can … #46

Merged
merged 1 commit into from
May 6, 2018
Merged

Convert the telegraf_plugins_extra varaible to a hash so that we can … #46

merged 1 commit into from
May 6, 2018

Conversation

tjend
Copy link
Contributor

@tjend tjend commented May 6, 2018

Convert the telegraf_plugins_extra varaible to a hash so that we can merge the hashes defined in multiple var files.

@dj-wasabi dj-wasabi merged commit 8f5cc09 into dj-wasabi:master May 6, 2018
@dj-wasabi
Copy link
Owner

Thanks! 👍

@gaelL
Copy link
Contributor

gaelL commented May 7, 2018

👎 this PR is not fully compatible with Telegraf config https://github.com/influxdata/telegraf/blob/master/docs/CONFIGURATION.md#multiple-inputs-of-the-same-type

Related issue : #48

gaelL added a commit to cycloid-community-catalog/stack-prometheus that referenced this pull request May 9, 2018
gaelL added a commit to cycloid-community-catalog/stack-external-worker that referenced this pull request May 9, 2018
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.

None yet

3 participants