tokkee.org

Merge branch 'collectd-4.2' into collectd-4.3

tcpconns plugin: Fix for compilation under FreeBSD.

rrdtool plugin: Simplified heartbeat calculation

Looks simpler, should do the same.

Signed-off-by: Florian Forster <octo@huhu.verplant.org>

hddtemp plugin: Use SCSI_DISK8_MAJOR thru SCSI_DISK15_MAJOR only if available.

Some Linux versions don't seem to provide the major numbers 8 thru 15.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge branch 'collectd-4.2' into collectd-4.3

ping plugin: Fixed a typo when setting the TTL.

PING_DEF_TIMEOUT used to be used instead of PING_OPT_TTL as option
argument of ping_setopt(). This caused really strange effects when pinging
hosts, like a failed assertion in ping_timeval_sub() in liboping
indicating a latency <= 0.

Thanks to Romain Francoise for pointing this out and a lot of helpful
debugging information.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

exec plugin: Don't pass an invalid egid to setgroups() if no group was given.

In case that no group has been specified in the config file, egid = -1 used to
be passed to setgroups() which, obviously, is not what we want.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

perl plugin: Warn if "EnableDebugger" has been used after "LoadPlugin".

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge branch 'collectd-4.2' into collectd-4.3

Merge branch 'collectd-4.1' into collectd-4.2

ping plugin: Deactivate the plugin if (pingobj == NULL).

If (pingobj == NULL), then the init-function will return (-1) so that the
plugin is deactivated.

src/collectd.c: Fix the arguments passed to an error message.

Unfortunately the compiler doesn't check va-args, so this was unnoticed :/

Merge branch 'collectd-4.2' of octo@verplant.org:/var/lib/git/collectd into collectd-4.2

swap plugin: Reapply a patch for Solaris.

This is a previously applied change that somehow got lost.

See http://git.verplant.org/?p=collectd.git;a=commitdiff;h=ee1113650b0b565776fc3d62162d84741e234f05
and http://git.verplant.org/?p=collectd.git;a=commitdiff;h=f68b68c5825efbd23c51531ff4e43c78a3bb2b3b

contrib/extractDS.px: Enhanced the script to modify RRD files a bit.

It can now also change the stepsize and add an RRA.

ChangeLog: Removed the legacy message about Nagios integration in the exec plugin.

README: Moved "libxml2" from the "Features" section to "Prerequisites".

D'oh!

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge branch 'pull/collectd-4.1' into collectd-4.1

src/collectd.c: Write a more informative error message if looking up the FQDN fails.

As requested by Micha Krause.

exec plugin: Use `setgroups' to set the list of supplementary group IDs.

Thanks to Peter Holik for suggesting this.

src/configfile.c: Fix another access of free'd memory.

This led to a crash with the default config of the Debian package.

liboconfig: Make sure that option-strings can always be freed.

This resolved a nasty crash when including files.

Bumped version to 4.3.0; Updated ChangeLog.

exec plugin: Removed the Nagios logic from the plugin.

Instead point users to the `exec-nagios.px' script in collectd-exec(5).

migrate-3-4.px: Do not translate the disk names used by hddtemp.

In version 4, hddtemp still uses major/minor device numbers to identify disks.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

migrate-3-4.px: Do not translate the disk names used by hddtemp.

In version 4, hddtemp still uses major/minor device numbers to identify disks.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge branch 'collectd-4.2'

Merge branch 'collectd-4.1' into collectd-4.2

Merge branch 'sh/hddtemp'

hddtemp plugin, collectd.conf(5): Added "TranslateDevicename" config option.

If enabled, translate the disk names to major/minor device numbers. For
backwards compatibility this defaults to true but it it recommended to disable
it as it will probably be removed in the next major version. Any other parts
of collectd have changed in a similar way with version 4.0, so hddtemp should
do so as well.

This option is disabled in new versions of the config file.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

rrdtool plugin: Sort the list of user defined `RRATimespan's.

This is required by the logic calculating the RRA sized.

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

perl plugin: Do not abort while preparing the stack for a subroutine call.

Else the stack would probably be left in an inconsistent case and might even
leak memory.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

collectd.conf(5): Added a note that libvirt's RefreshInterval may be disabled.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

README: Added libxml2 to the list of prerequisites.

The libvirt plugin uses this library to parse data provided by libvirt.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

README: Fixed a typo.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Updated collectd.conf.in.

 * Missing plugins and config options have been added.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

contrib/README: Removed the section about PerlLib/.

This subdirectory no longer exists - it has been moved to bindings/perl/.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Makefile.am: Create `$(localstatedir)/log' so the logfile plugin works out-of-the-box.

And don't search for `.svn' directories anymore - they're long gone.

exec plugin: Changed the format in which notifications are passed to the programs.

Using a `Message:' header field restricts us to one-line messages, which is
probably not very farsighted.

exec plugin: Disabled the `NagiosExec' option.

All references to this option have been removed from the manpages, too.

collectd.conf(5), ChangeLog: Documented the new `Include' features.

src/configfile.c: Use wordexp(3) to expand shell wildcards if available.

contrib/migrate-3-4.px: Handle `hddtemp' graphs correctly.

contrib/exec-nagios.px: Added a Perl script which handles Nagios plugins.

It handles most of the Nagios plugin API, including multi-line output (a
Nagios 3 feature) and performance data. It's basic on purpose so that easy
stuff is possible but complex stuff is about as hard as rewriting the plugin ;)

contrib/exec-munin.px: Fix a typo in the config handling functions.

src/utils_ignorelist.c: Remove an annoying debug message.

configure: Fix a syntax error.

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

configure: Correctly handle all cases when using AC_ARG_WITH().

In a lot of cases only "$withval" != "yes" and "no" had been handled when the
option had been given. Therefore, the option had been ignored if no argument
had been passed to the option, making --with-<option> and / or
--without-<option> a no-op. This patch fixes this issue.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

contrib/examples/: Added an example for a notification callback.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

collectd(1), README: Document notifications and related plugins.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

collectdmon(1): Added a section about supported signals.

Documented the effects of the SIGINT, SIGTERM and SIGHUP signals.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

README: Added "uuid" plugin to the list of available plugins.

 * Added an optional dependency on libhal.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

README: Updated package description.

Added a note about the monitoring support (this has been copied from the
Debian package description).

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

src/configfile.c: Extend the `Include' statement to handle directories, too.

Directories are included recursively. Files and directories begnning with a dot
are ignored. Symlinks are followed. You can crash collectd by looping the
symlinks.

configure: Warn if librrd is not thread safe.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

plugin.c: Use an AVL tree to store the data sets.

The list of data sets is queried quite frequently but hardly ever modified.
Using an AVL tree instead of a linked list improves the search time from O(n)
to O(log n).

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

exec plugin: Fixed a possible (though very improbable) memory leak.

In case of an error, the program_list_and_notification_t pointer has not been
freed before calling pthread_exit() from exec_notification_one().

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Collectd.pm: Use $interval_g instead of a hard-coded value.

The hard-coded value was still legacy code from the time before the global
variables have been exported to Perl.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

perl.c: Unregister notifications in perl_shutdown().

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

perl plugin: Fixed a bug in Collectd::plugin_unregister_data_set().

The function tried to access a nonexistent second argument.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

collectd-perl(5): Added documentation for the notification support.

Some minor errors have been fixed as well and the ChangeLog has been updated.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

perl plugin: Exported plugin_dispatch_notification() to Perl.

This adds the following function to collectd's Perl API:

Collectd::plugin_dispatch_notification:
  submit a notification to collectd's notification mechanism

  arguments:
  notif - notification

A notification is a reference to a hash with the following members:
  severity => $severity (default: NOTIF_FAILURE)
  time     => $time (default: time (NULL))
  message  => $msg
  host     => $host (default: hostname_g)
  plugin   => $plugin
  type     => $type
  plugin_instance => $instance
  type_instance   => $type_instance

The severity should be any of the Collectd::NOTIF_* constants.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

perl plugin: Added support to dispatch notifications to Perl plugins.

Perl plugin may now register a callback of type Collectd::TYPE_NOTIF which
accepts a single hash argument with the following members:

  severity
  time
  message
  host
  plugin
  plugin_instance
  type
  type_instance

Severity may be any of the following values (exported by the "notif" tag):

  Collectd::NOTIF_FAILURE
  Collectd::NOTIF_WARNING
  Collectd::NOTIF_OKAY

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

ChangeLog: Describe changes to the perl plugin in more detail.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

configure.in: Fixed handling of the --with-perl-bindings option.

This option could not be disabled so far as this case has not been handled in
the configure script. This patch fixes this issue.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge branch 'collectd-4.2'

Merge branch 'collectd-4.1' into collectd-4.2

network plugin: Have two different threads for handling incoming packets.

One that only receives and enqueues packets and one which parses the packets
and dispatches them to the daemon. This should solve problems with (too) short
socket buffers and (very) heavy load.

Collectd::Unixsock: Added license information.

The copyright information in some changed files has been updated, too.

TODO: Removed the remark about the `libvirt' plugin.

Now it builds flawlessly. Apparently it was some leftover file in my build tree
that caused this weird problem.

TODO: Updated the todo file.

src/utils_threshold.c: Implemented `warning' and `failure' thresholds.

The options `Min' and `Max' have been renamed to `FailureMin', `FailureMax',
`WarningMin', and `WarningMax'.

logfile plugin: Only print the host field (and other fields) of a notification if they hold any information.

exec plugin: Implemented the `PUTNOTIF' command for external applications, too.

Also changed the manpage so that lines without the `PUTVAL' are mentioned as
being deprecated.

Collectd::Unixsock: Added the `putnotif' method to dispatch notifications.

Merge branch 'collectd-4.2'

Merge branch 'collectd-4.1' into collectd-4.2

collectd-unixsock(5): Copied Sebastian's additions from collectd-exec(5).

collectd-unixsock(5): Documented the PUTNOTIF command.

Merge branch 'collectd-4.1' of octo@verplant.org:/var/lib/git/collectd into collectd-4.1

src/common.h: Fixed many typos in the header file.

unixsock plugin: Imported the new `putnotif' command from the appropriate module.

src/utils_cmd_putnotif.[ch]: Added a new module which handles parsing of notifications from some command line or similar.

Of course, this is similar to the `utils_cmd_putval' module.

TODO: Updated the todo file.

Added the goal to remove the cache from the unixsock plugin.

src/liboconfig/parser.y: Updated copyright information.

src/{common,network}.[ch]: Updated copyright information.

TODO: Adjusted todo list.

network plugin: Added support for sending and receiving notifications.

Notifications are sent ``out of band'', i. e. each notification is one packet
and they are not mixed with performance data values.

Right now there is no configuration option to prevent listening sockets from
receiving notifications nor to prevent sending sockets from sending
notifications. This may be an option for the future, though.

src/utils_threshold.c: Check that the <Threshold> block does not have any arguments.

liboconfig: Allow the argument list of blocks to be empty.

This way the `<Threshold>' block doesn't need a bogus argument just to make the
parser happy.

collectd: Added ``associative'' members to the notification_t structure.

A notification does not need to be related to any performance data collected by
collectd, but now it *may* be related. This is used in the threshold stuff
already and passed via ExecNotification in the exec plugin, too.
The new `notification_init' function in `src/common.c' and the
`NOTIFICATION_INIT_VL' macro simplify the initialization of these new fields.

collectd.conf(5), collectd-exec(5): Documented the `NotificationExec' and `NagiosExec' options.

ChangeLog: Documented the new features in the master branch so far.

Also extended the TODO list a bit more, as things came to mind.

AUTHORS: Added Richard Jones.

Fixed some compiler warnings identified by gcc's -Wextra option.

The following issues have been addressed:

 * comparison between signed and unsigned - this was found in several places
   throughout the code and has been fixed in various ways
 * missing initializer - an incomplete initializer has been used for two
   struct instances in perl.c
 * unused parameter - when applicable, the parameter has been removed; in
   thirteen cases the parameter is required by different library API's and in
   two cases the parameter was left in place to retain a consistent interface
   within the affected modules; as __attribute__((unused)) is a GNU extension,
   it has not been used to document those exceptions

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

apache plugin: Improved some error messages and fixed-buffer string copying.

Return an error when initialization fails. This includes the case when no URL
is configured. The user is informed about this as clear as possible.

Also, `strncpy' and `snprintf' now are followed by a forced NULL-termination of
the strings to be on the save side. This also changes the plugin to look more
like the other plugins which mostly do this already.

apache plugin: Do not initialize the plugin if no URL has been specified.

There is no need to initialize curl, if the plugin will not be used anyway.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge branch 'pull/master'