Description of problem: We need to add a file for each plugin loaded by collectd which we configure from the metrics side. This will make configuration of collectd easier, and plugins easier to navigate and enable/disable. Based on BZ#1471949 comment#6 Version-Release number of selected component (if applicable): ovirt-engine-metrics-1.1.1-0.0.master.20171001113530.el7.centos.noarch How reproducible: 100% Steps to Reproduce: 1. Check configuration files on collectd in /etc/collectd.d/ Actual results: 20-builtins-conf-for-ovirt-engine.conf Expected results: 20-ovirt-postgresql.conf 20-ovirt-cpu.conf 20-ovirt-memory.conf 20-ovirt-interface.conf
This has the advantage of a more organised conf.d and might allow an easier way for the user to figure out what plugins are loaded. But the disadvantage is that if a certain plugin should not be used anymore, then we should remove the file. That means adding another ansible role that deletes the required files. Currently, for most of the plugins we only require updating 1 file.
Can you please explain why we need to remove the file? I don't see any harm in just disabling the plugin inside the file and leaving the configuration untouched as admin can re-enable it later, or on demand.
If the purpose is for a better user experience, to figure out what plugins are loaded, having files with disabled plugins will only add confusion and make this harder for the user to get the full list. This RFE, in my opinion, is only a good to have.
Not really critical in my opinion, it is common to have a single system conf.