tokkee.org

email plugin: Unlock mutexes before submitting the data.

All values are copied to local data structures in email_read () before they
are submitted. Thus the mutex can be unlocked before data is sent to the rrd
files or the network.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

Merge branch 'collectd-3.10'

Conflicts:

ChangeLog

Bumped version to 3.10.4; Updated ChangeLog.

Bug#404018: Close the file descriptor when binding to a socket fails.

This closes the second part of the bug reported by Max Kellermann.

Bug#404018: Break out of the `while (loop)'-loop when no socket can be opened.

This fixes the first part of the bug reported by Max Kellermann.

New plugin "mbmon" to collect motherboard status information:
temperatures, voltages and cooling fan speeds.

Signed-off-by: Flavio Stanchina <flavio@stanchina.net>

email plugin: Made read_{char,line} functions static.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

email plugin: Write "0" values as well to distinguish from NaN's.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

collectd{,.conf}.pod: Extended and clarified documentation of the email plugin.

* Added a description of the protocol and the DS's to the manpage.
* Make clear that the group of the UNIX socket is always changed when running
  as root.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

sensors plugin: Simplify the reload machanism.

Instead of calling `stat' in the `read' function and then, eventually, calling
the reload stuff, just call the reload function every time and have it return
without doing anything if the config didn't change. This way the reload logic
is not in the read function.

Merge branches 'sh/email' and 'ls/processes'

sensors plugin: Reload lm_sensors' configuration when changed

Signed-off-by: Stanek Lubos <kolektor@atlas.cz>

sensors plugin: Made sensors.conf path configurable

Signed-off-by: Stanek Lubos <kolektor@atlas.cz>

sensors plugin: syslog() and DBG() calls unification

Signed-off-by: Stanek Lubos <kolektor@atlas.cz>

processes plugin: Do not complain of a disappeared task's proc dir

Signed-off-by: Stanek Lubos <kolektor@atlas.cz>

email plugin: Prepend the module name to any syslog messages

Signed-off-by: Sebastian Harl <sh@tokkee.org>

email plugin: Kill threads before closing the associated socket

If the socket is closed before the thread is terminated it might cause some
confusing error messages if the thread wants to read from the socket.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

SA plugin: Documentation updates

Signed-off-by: Alexander Wirt <formorer@formorer.de>

SA plugin: Added reconnect support (collectd_retries), code cleanup

The plugin now tries to reconnect if connection isn't possible
(default is 3 tries), this can adjusted with the collectd_retries
config parameter.

Signed-off-by: Alexander Wirt <formorer@formorer.de>

src/Makefile.am: Added utils_dns.h to dns_la_SOURCES.

Without this change the .tar.{bz2,gz} does not contain utils_dns.h.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

contrib/collection.cgi: Added support for the email plugin.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

email plugin: Do not allocate memory for the return value in read_line ().

The read_line () function now uses the provided buffer to save the return
value to. In case no complete line could be read, read_line () is called
recursively until '\n' has been found or end-of-file has been reached.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

email plugin: Use a thread pool.

Up to now the email plugin created a thread for each connection. This thread
was destroyed after the connection has been closed. Now, a pool containing
MaxConns threads is created and a new connection is assigned to a free thread.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

email plugin: Prevent a segfault in read_line ().

Signed-off-by: Sebastian Harl <sh@tokkee.org>

SpamAssassin plugin: Reclicense again und remove debugging die

Due to some misunderstanding Artistic was not the right license so
I should GPL now instead of Artistic.
Additionally I removed a debugging "die" that should never have been checked
in into my svn repo (Thanks tokkee).

Signed-off-by: Alexander Wirt <formorer@formorer.de>

SpamAssassin plugin: Update license foo and add timout

Also updated example.cf to reflect the new timeout option
(default is 2 sec.). The whole thing is now dual licensed
(Artistic and Apache License).

Signed-off-by: Alexander Wirt <formorer@formorer.de>

load plugin: Made the data source variable `static'.

Also removed the `filename' member from `data_set_t' because it's not used.

load plugin: Converted to the new plugin structure.

rrdtool plugin: First implementation of an rrdtool write plugin.

Many functions have been moved from `common.c' to a new file `rrdtool.c'. The
new plugin compiles, but has not yet been testet.

plugins: Implement a first version of the new plugin mechanismn.

Instead of the impractical `plugin_register' function, provide a variety of
different register functions, so plugin can `hook in' at various stages of the
daemon.

The most important new hook is likely the `write' hook which will allow for
generic `output plugins' to be plugged in.

src/utils_llist.[ch]: Added a generic linked list implementation.

email plugin: Added some information about this plugin and it's config options to the manpages.

Also updated the README file, fixed a little indentation in `src/collectd.h'
and force the base of the `SocketPerms' to be eight (i. e. assume octal numbers
when setting the permission).

Merge branch 'master' of octo@verplant.org:/var/lib/git/collectd

dns plugin: Documented plugin in the README file and the manpages.

contrib/collection.cgi: Added graphs for `opcode' RRD-files (from the dns plugin)

src/utils_dns.c: Implemented a handler for `DLT_LINUX_SLL'.

This is the ``Linux cooked capture encapsulation'', which is at least returned
when capturing the `any' device under Linux. This patch will strip off the
header and pass the packet to the IPv4 or IPv6 handler, whichever one is
ppropriate.

Merge branch 'aw/spamassassin'

Merge branch 'ls/shutdown'

sensors plugin: declare variables only if SENSORS_HAVE_READ

Signed-off-by: Stanek Lubos <kolektor@atlas.cz>

Merge branch 'ls/shutdown'

sensors plugin: fix shutdown for SENSORS_HAVE_READ

Sorry, too many changes for me to observe at once.

Signed-off-by: Stanek Lubos <kolektor@atlas.cz>

sensors plugin: shutdown implementation

Signed-off-by: Stanek Lubos <kolektor@atlas.cz>

sensors plugin: SENSORS_HAVE_READ unification

Signed-off-by: Stanek Lubos <kolektor@atlas.cz>

Merge branch 'sh/shutdown'

email plugin: Shutdown cleanly.

Using a shutdown function the email plugin closes any sockets, terminates all
running threads and removes the UNIX socket.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

email plugin: "Calculate" UNIX_PATH_MAX if it is not defined.

Some systems (e.g. Darwin) do not seem to define UNIX_PATH_MAX at all. Thus I
calculate it from the size of the sun_path character array of struct
sockaddr_un in this case.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

email plugin: Make UNIX socket path configurable at compile time.

The prefix for the UNIX socket can be configured using the
COLLECTD_SOCKET_PREFIX macro. "email" will be prepended to it.

If you configure collectd using

  CFLAGS="-DCOLLECTD_SOCKET_PREFIX='\"$path-\"'" ./configure

the socket will be found at "$path-email".

Signed-off-by: Sebastian Harl <sh@tokkee.org>

dns plugin: Remove unnecessary poll code and use the (blocking) `pcap_loop' function instead.

Not only does this simplify things *a lot* on *BSD and Mac OS X, because you
can't always call `poll' on their filedescriptors, it also simplifies the code
a lot and thus removes potential error sources.

dns plugin: Use the standard member names for `struct in6_addr' accesses.

The 32bit members apparently are Linux or glibc specific, so don't use them.

Imported Alexander's SpamAssassin-plugin to contrib/SpamAssassin/

configure.in: Set the default case for `--with--libpthread' to `yes'.

Also removed the `regex' output at the end of the configure run, because it was
empty.

plugin.[ch]: Added support for an optional plugin shutdown function.

A plugin may register a shutdown function using plugin_register_shutdown ().
This function is called when collectd terminates either during normal
termination or after SIGINT or SIGTERM have been received.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

dns plugin: Fixed a compile error if pcap.h is not found.

If pcap.h is not found, the compiler complains about some "defined but not
used" warnings. As we're using -Werror this causes the build to abort.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

ChangeLog: Collected the changes so far.

The new version needs to be released soon!

Merge branch 'ff/processes'

email plugin: Limit the `MaxConns' option by a hardcoded value.

Because typos (and ``typos'', i. e. dumb users) happen, it's better to not
allow INT_MAX connections. The problem is that on 32bit machines this would
 a) create 2147483648 threads
 b) allocate (at least) 512 GBytes of memory
which would result in certain death of either the daemon or the system.

This patch limits the number of connections (and thus threads and allocated
memory) to 16384, which ought to be enough for most people. Those, who need
more connections (and can accomplish this, even though there are quite narrow
OS limits) will need to recompile themselves.

email plugin: Made socket settings configurable.

Added config file support to the email plugin.

The following options are available:
* "SocketGroup <group name>"
  Set the group the UNIX socket belongs to to <group name>.
* "SocketPerms <perms>"
  Set the permissions of the UNIX socket to <perms>. No validation is done.
  The user has to make sure reasonable values are given.
* "MaxConns <conns>"
  The maximum number of concurrent connections is set to <conns>.

<perms> and <conns> may be given as decimal (no prefix), octal (prefix "0") or
sedecimal (a.k.a. hexadecimal, prefix "0x") values.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

src/utils_avltree.c: Fix and endless loop. Added more `assert's.

It's broken due to the asserts right now, but this only means I haven't found
another bug yet.

email plugin: Make it possible to build the plugin write-only.

So far the plugin simply imported `pthread.h'. If this headerfile (or the
library for that case) was not present, building the plugin would fail. This
patch makes `libpthread' a soft dependency, i. e. it's only necessary if you
want to read from the socket. Without `libpthread' being available the plugin
can still be used in server processes.

Merge branches 'ff/dns' and 'sh/email' into next

Conflicts:

configure.in
src/Makefile.am

dns plugin: Improved config checks for the pthread library.

The checks for the pthread-library have been extended to match the other
libraries being used. The dns plugin (in particular `utils_dns.c') has been
verified to build without `libpcap' being present.

dns plugin: Added the dns plugin to the sample `collectd.conf'.

email plugin: Use strtok_r() instead of strtok().

strtok() internally uses a static buffer and thus is not thread-safe.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

email plugin: Change owner and mode of the UNIX socket if possible.

Set the owner and mode to COLLECTD_GRP_NAME (defaulting to "collectd") and 0770
respectively. A couple of different daemons running as different users might
want to connect to it.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

email plugin: Minor code cleanup.

Removed debugging output and whitespaces at end of lines.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

src/utils_ignorelist.c: Fix build-problems when building without regex support.

dns plugin: Check for `libpthread' and link the plugin against it.

dns plugin: Use `pthread_sigmask' rather than `signal'.

dns plugin: Use threads rather than two processes and a pipe.

This is not only much simpler, code wise, but also more elegant and hopefully
faster. This code is alpha quality at best.

New plugin "email" to collectd ham, spam, ... statistics

This plugin collects email count and size for each type (e.g. ham, spam,
virus, ...) of emails, spam score values and the count of successful spam
checks (e.g.  BAYES_99, SUBJECT_DRUG_GAP_C, ...).

These information are provided by external programs which communicate with the
plugin thru a UNIX socket and a simple line-based protocol:

/* e-mail type (e.g. ham, spam, virus, ...) and size */
e:<type>:<bytes>

/* spam score */
s:<value>

/* successful spam checks */
c:<type1>[,<type2>,...]

At most MAX_CONNS (currently set to 5) clients can connect to the plugin
simultaneously. Each connection is handled by a separate thread. Any input
line is limited to 256 characters (including the newline character) which
ought to be enough for anybody[tm] by definition. "c"-lines have to be split
up if they grow longer.

Signed-off-by: Sebastian Harl <sh@tokkee.org>

configure.in: Removed `src/libconfig' from the list of subdirs requireing configuration.

src/Makefile.am: Link the collectd binary with `src/utils_avltree.[ch]'.

src/utils_avltree.[ch]: Changed the interface to return the key upon deletion.

The idea behind this is, that the calling routing may not have a pointer to
it's location, and possibly can't deallocate the memory.

src/utils_avltree.c: Fixed the first few bugs.

I a module like this, bugs are to be expected ;)

Added src/utils_avltree.[ch]: An implementation of an AVL-tree.

Merge branch 'ff/dns'

Conflicts:

configure.in
contrib/collection.cgi

df plugin: Match `spec_device' rather than `device' when matching the `Device' ignorelist.

src/Makefile.am: Install the `collectd.conf' with restricted permissions, 0640.

Thanks to Lubos for suggesting this :)

df plugin: Fix a nasty bug in the config routine. It effectively disabled the `IgnoreSelected' option.

sensors plugin: Declare static arrays only if they're being used, i. e. if libsensors exists.

processes plugin: Implemented detailed process statistics for Darwin/Mac OS X

collectd.conf(5): Documented the new selection options of the df-plugin.

configure.in: Remove the `--with-regex' option and check for `regex.h' everywhere.

df plugin: Make use of the new ignorelist functionality.

src/utils_ignorelist.c: Allow the `entry' argument to be NULL/empty.

Also assert the object to be non-empty.

src/utils_ignorelist.[ch]: Remove the `ignorelist_num' function and associated code.

src/utils_ignorelist.[ch]: Renamed some functions. Internal changes.

Mostly renamed variables, but also changed some return values and inserted
asserts where appropriate.

Ignorelist: Renamed `src/config_list.[ch]' to `src/utils_ignorelist.[ch]'.

The interface has been changed to use the `ignorelist'-prefix rather than
`configlist', which is a confusing term for this functionality.

More improvementa on the ignorelist functionality.

I am sending the updated version of config_list.
I repaired the error freeing unallocated memory, modified regex for dynamically
allocated error message buffer and replaced '|' with '/'.

Improve and generalize the ignorelist functionality.

Change contrib/init.d-rh7 to sleep for a second between restarts.

Break out of the `nanosleep' loop if `loop' is set to non-zero.

plugin_read_all: Check the `loop' variable after each iteration and return early if the daemon is shutting down.

Lubos Stanek has reported that the daemon may stop too slow, resulting in it
being killed by impatient init-scripts. With this patch `plugin_read_all'
checks the `loop' variable before entering each plugin's read-function. If the
loop-variable is set to non-zero it will return early, allowing the daemon to
shutdown more instantly.

sensors plugin: Improved the documentation in the sample configfile and the `collectd.conf(5)' manpage.

sensors plugin: Some code cleanup.

Very long lines have been broken into several shorter ones. `strncmp' has been
replaced with `strcmp' where reasonable. Variables have been renamed and all
variable declarations have been moved to the top of the block, as required by
ANSI C prior to C99.

Merge branch 'master' into ls/sensors

Merge branch 'processes'

Merge branch 'master' into ff/dns

dns plugin: Corrected the handling of signals, i. e. don't catch them.

Merge branch 'collectd-3.10'