network plugin: Interface option moved to Server/Listen block
authorMax Henkel <henkel@gmx.at>
Fri, 2 Apr 2010 10:41:11 +0000 (12:41 +0200)
committerFlorian Forster <octo@huhu.verplant.org>
Mon, 5 Apr 2010 13:37:21 +0000 (15:37 +0200)
commit0df36253a21b2eec9784bb6a15ee564559f145c4
tree129d14784ef04f0bbac1cf61e3248b2a90599433
parent9d095a155631f4cf973b9c5a489f81ffbef5263b
network plugin: Interface option moved to Server/Listen block

Hello list!

On Tue, Mar 09, 2010 at 06:17:35PM +0100, Florian Forster wrote:
[...]
> On Fri, Feb 26, 2010 at 12:49:02PM +0100, Max Henkel wrote:
[...]
> > @@ -2842,6 +2929,8 @@ static int network_config (oconfig_item_t *ci) /* {{{ */
> >        network_config_add_server (child);
> >      else if (strcasecmp ("TimeToLive", child->key) == 0)
> >        network_config_set_ttl (child);
> > +    else if (strcasecmp ("Interface", child->key) == 0)
> > +      network_config_set_interface (child);
> >      else if (strcasecmp ("MaxPacketSize", child->key) == 0)
> >        network_config_set_buffer_size (child);
> >      else if (strcasecmp ("Forward", child->key) == 0)
>
> Starting with the signing and encryption option, we have <Server> and
> <Listen> blocks that take socket specific options. It'd be kind of
> inconsistent with the "TimeToLive" and other options, but maybe adding
> this to those blocks would be even better. It would enable to forward
> data from one multicast group on interface-0 to the *same* multicast
> group on interface-1, which I guess could be useful. But maybe it's an
> academic case, I dunno.. Thoughts, anyone?
[...]

This patch provides the configuration possibility inside the
Server/Listen-block, but I'm unsure if the "int interface" is
located properly inside the sockent_t.

Best regards,

Max

Signed-off-by: Florian Forster <octo@huhu.verplant.org>
src/collectd.conf.pod
src/network.c