Show default settings in mysql sample config#6484
Conversation
| # gather_table_schema = false | ||
|
|
||
| ## gather thread state counts from INFORMATION_SCHEMA.PROCESSLIST | ||
| gather_process_list = true |
There was a problem hiding this comment.
Wouldn't this (and others) negatively affect those generating and using the config file?
There was a problem hiding this comment.
Can you give an example of what might happen?
There was a problem hiding this comment.
This changes the default/generated config file, so if anyone is relying on these values, telegraf will no longer do what they are expecting. In this case, the process list would no longer be gathered unless manual action is taken on their part, whereas before no action was required.
There was a problem hiding this comment.
I see, a newly generated config would be different from a generated config in the previous version. The reason I think this is okay is that previously generated configs will continue to work as before. The default values aren't changing (with the exception of the values in the first comment) they were documented incorrectly.
The sample config now shows the default values. I added new defaults for the perf events since the existing defaults were not usable and the plugin required them to be set.
closes #6444
Required for all PRs: