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.

Sign up for GitHub

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

Jump to bottom

Cleanup config file #948

Merged
merged 1 commit into from
Apr 1, 2016
Merged

Cleanup config file #948

merged 1 commit into from
Apr 1, 2016

Conversation

sparrc
Copy link
Contributor

@sparrc sparrc commented Mar 31, 2016

No description provided.

@sparrc sparrc force-pushed the conf-cleanup branch 2 times, most recently from dc4f87b to 4035bca Compare April 1, 2016 00:53
@sparrc
Copy link
Contributor Author

sparrc commented Apr 1, 2016

cc @pauldix it took ~200 years, but this will close #199, see https://github.com/influxdata/telegraf/blob/conf-cleanup/etc/telegraf.conf for what the default telegraf config file would look like going forward

@sparrc sparrc force-pushed the conf-cleanup branch 2 times, most recently from 3779116 to ee74f8c Compare April 1, 2016 02:46
@sparrc sparrc mentioned this pull request Apr 1, 2016
Closed
@sparrc sparrc force-pushed the conf-cleanup branch 3 times, most recently from 10658cb to 2e75535 Compare April 1, 2016 16:12
@sparrc
Cleanup & standardize config file
e436b2d 
changes:

- -sample-config will now comment out all but a few default plugins.
- config file parse errors will output path to bad conf file.
- cleanup 80-char line-length and some other style issues.
- default package conf file will now have all plugins, but commented
  out.

closes #199
closes #944
@pauldix
Copy link
Member

pauldix commented Apr 1, 2016

👍

@bai
Copy link

bai commented Apr 1, 2016

Awesomeness, can't wait.

@sparrc sparrc merged commit e436b2d into master Apr 1, 2016
@sparrc sparrc deleted the conf-cleanup branch April 1, 2016 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@sparrc @pauldix @bai