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

Optional argument to ignore bad configs #4729

Closed
absolutejam opened this issue Sep 20, 2018 · 3 comments
Closed

Optional argument to ignore bad configs #4729

absolutejam opened this issue Sep 20, 2018 · 3 comments
Labels
area/agent feature request Requests for new plugin and for new features to existing plugins

Comments

@absolutejam
Copy link

Feature Request

Proposal:

Argument to allow Telegraf to ignore bad config files instead of throwing a critical exception.

Current behavior:

Currently, a bad telegraf config (Within --config-directory) will kill the Telegraf agent.

Desired behavior:

Option to ignore bad config file and log an message instead.

Use case: [Why is this important (helps with prioritizing requests)]

I didn't see a way to validate configs (Did I miss this), and at the moment, a badly rendered config file from SaltStack causes the entire service to fail. I'd like to ignore bad configs and allow metrics collection to continue instead of having a blackspot in our collection.

On a side note, is there a way to validate configs?

Cheers!

@absolutejam
Copy link
Author

Actually, it looks like the --test param can validate the config, but I'd still like this option to be considered 😄

@danielnelson danielnelson added the feature request Requests for new plugin and for new features to existing plugins label Sep 21, 2018
@danielnelson
Copy link
Contributor

I think this would be a good feature, maybe not for the agent section but at least for plugins, and I especially like the idea of an option to validate the config.

Here is a related issue about Telegraf not starting when an ouput connection cannot be made: #3723

@sspaink
Copy link
Contributor

sspaink commented Apr 18, 2022

Closing in favor of: #5507
I think adding a way to validate configs is a good idea.

@sspaink sspaink closed this as completed Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/agent feature request Requests for new plugin and for new features to existing plugins
Projects
None yet
Development

No branches or pull requests

3 participants