[collectd] New aggregator plugin "basic_aggregator" (#136)

[Sebastian Harl]

Hi,

On Fri, Oct 12, 2012 at 04:26:28PM +0200, Fabien Wernli wrote:
> On Fri, Oct 12, 2012 at 02:46:00PM +0200, Yves Mettier wrote:
> > For the step 2/ I suggest a new command of collectdctl. That command
> > would allow to specify the plugin to reconfigure, or something like
> > "all" to reconfigure all the plugins. Does that sound correct ?

Right, that's what I was thinking about. collectdctl uses the UNIXSOCK
interface to communicate with collectd. So after implementing my
proposed change for that, collectdctl could be updated to support the
new command as well.

> Adding support for signals would also be nice, e.g. SIGHUP, which
> people are very familiar with

Ack. I'm not sure, though, if it's a good idea to implement that before
most/many/all parts of collectd actually support "reconfigure" as people
might expect too much from SIGHUP else.

Before adding support for SIGHUP, a reload could be done through the
UNIXSOCK mechanism. This would allow the daemon (and the caller) to
directly display any (error) messages to the user. This can also be
implemented in an init script (or similar), so the user will get
(another) very familiar interface (with the side-note that the
'unixsock' plugin will have to be enabled).

Thoughts about that?

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://mailman.verplant.org/pipermail/collectd/attachments/20121013/34d1e5f4/attachment.pgp>