Hi,

On Tue, Oct 16, 2012 at 11:31:17AM +0200, Laurent Bigonville wrote:
> It could be nice if you could add a "/etc/collectd.d" directory where
> the user could drop config stanza to easily enable plugins.

Sounds like a good idea to me.

> Adding Include "/etc/collectd.d/*.conf" in collectd.conf and creating
> the directory seems to work.

Well, kind of ;-) This will report an error on startup if there is no
file matching the specified pattern. This would confuse users if it was
used in the default configuration (-> everybody would get the error).

Anyway, I've created a patch adding support for recursively including
all files matching a specified pattern. In this case, no error will be
generated if there is no match. This way, I could add the following to
collectd.conf:

  Include "/etc/collectd/collectd.conf.d" "*.conf"

This would work similar to "find /etc/collectd/collectd.conf.d -name
*.conf" and include all files found this way.

See <https://github.com/collectd/collectd/pull/205> for details.

What do you think? ;-)

Cheers,
Sebastian

-- 
Sebastian "tokkee" Harl +++ GnuPG-ID: 0x8501C7FC +++ http://tokkee.org/

Those who would give up Essential Liberty to purchase a little Temporary
Safety, deserve neither Liberty nor Safety.         -- Benjamin Franklin

Attachment: signature.asc
Description: Digital signature

Reply via email to