tokkee.org

Merge branch 'collectd-4.6'

rrdtool plugin: Beatify a debugging message.

Merge branch 'sh/collectd-4.5' into sh/collectd-4.6

Conflicts:
contrib/collection3/lib/Collectd/Graph/Common.pm
src/postgresql_default.conf

postgresql_default.conf: Make sure the "disk_io" query does not return NULLs.

Starting with some version between 8.3.3 and 8.3.6, pg_statio_*_tables returns
NULL instead of 0 for statistics if no instance of the appropriate relation
exists. PQgetvalue() returns an empty string in that case which would then
result in error messages when udb_result_submit() tries to convert that to a
number.

Now, the "disk_io" query uses PostgreSQL's coalesce() function to make sure 0
is returned instead of NULL.

src/plugin.c: Remove an unused variable.

contrib/exec-nagios.px: Make it possible to run the same script multiple times.

rrdtool plugin: Fix a possible race condition at startup.

On a very busy system, if the network plugin was initialized before the
rrdtool plugin, `rrd_cache_insert' may be called before the RRDtool
plugin is initialized. It would then pass `cache == NULL' to
`c_avl_get', resulting in a segmentation fault.

collectd.conf(5): Don't mix spaces and tabs in configuration examples.

contrib/collection3: Fix stacking with NAN values.

Stacked values above NAN values were not drawn previously..

contrib/collection3: Minor cleanups.

contrib/collection3: Add the "IgnoreUnknown" config option.

Used by the "GenericStacked" graphs.

contrib/collection3: Add "Total" to "GenericIO" graphs.

Using the RRDtool 1.2 VDEF syntax.

contrib/collection3: Add the "DataDir" config option.

contrib/collection3/bin/.htaccess: s/SetHandler/AddHandler/

SetHandler doesn't accept a file extension.

contrib/collection3/bin/.htaccess: Add file.

This has been missing. Wonder why nobody complained..?

postgresql_default.conf: Added a short usage and copyright header.

postgresql plugin: Made variable names more self-explanatory.

Also, while I was at it, improved some error messages.

postgresql_default.conf: Don't use the deprecated {Min,Max}PgVersion options.

collectd.conf(5): Mention that `Oracle' doesn't like semicolons..

.. at the end of the SQL statement.

libcollectdclient: Check if the file handle is valid before writing to it.

And free allocated memory when connecting fails. Only assign anything to
the user-provided pointer if `lcc_connect' succeeds. This should make
error-handling much more straight-forward.

snmp plugin: Added a note about a potential bug that needs fixing.

Negative absolute (gauge) values may be casted to an unsigned integer
variable on their way throught he daemon, producing VERY HIGH tempera-
tures in winter..

Merge branch 'bp/php-collection' into collectd-4.5

php-collection: Add support for named graph lists as favorites

php-collection: Add support for named graph lists as favorites

php-collection allowed saving and loading a list of graphs from a
cookie.
This patch changes loading/saving to provide support for naming
of favorite graph lists.
When saving a name has to be provided (up to 30 alphanumerical
characters includeing underscore and hyphen.
For loading a named list can be selected from a dropdown list (this list
is refreshed when it gets focus) of available favorites.

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

php-collection/functions.php: Escape/avoid problematic characters in RRD commandline

php-collection/functions.php: Escape/avoid problematic characters in RRD commandline

rrdtool does not like '!' in variable names so replace them with '_'.
(bind plugin has them in a few type-instances and type-instance is used
as variable name by MetaGraphs)
Also escape rrdtool field seperators that might appear in file names.

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

php-collection/definitions.php: Add MetaGraph definitions for bind plugin

php-collection/definitions.php: Add MetaGraph definitions for bind plugin

Add metagraph definition and matching function for the various
types used by bind plugin.
Adjust the memory MetaGraph function to support bind memory statistics.

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

php-collection/graph.php: Add missing error500() function

php-collection/graph.php: Add missing error500() function

When a call to rrdtool fails or some other internal error occurs
a HTTP error with code 500 and matching error image should be
returned, added the missing error500() function which does this
the same way as error404() and error400() functions.

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

php-collection: Fixes and multiple saved graph lists

php-collection/functions.php: Fix collectd_flush()

Some obsolete code in collectd_flush() caused PHP Notice errors
and prevented caused the function to return before attempting
to ask collectd to flush cache.

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

php-collection/definitions.php: Add missing MetaGraph for tcp_connections

php-collection/definitions.php: Add missing MetaGraph for tcp_connections

Add missing MetaGraph reference for tcp_connections and fix DS name
for the MetaGraph

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

contrib/php-collection: A few more bug-fixes to the graphing scripts.

This removes some obsolete code which prevents flush from working and adds
missing wrapper error500().

src/plugin.c, network, rrdtool: Improved thread shutdown.

Just some comments, info messages and not assigning zero to pthread_t.

ping plugin: Add support for drop rate and standard deviation.

README: Documented the `--with-fp-layout' configure option.

src/collectd.conf.in: Only include very few standard plugins by default.

Available but disabled by default plugins are listed with one hash
mark ("#") in front of them; unavailable plugins are listed with two hash
marks ("##") in front of them.

Plugins loaded by default are:

 - syslog || logfile
 - rrdtool || network || csv
 - cpu
 - interface
 - load
 - memory

protocols plugin: Add to configuration summary and collectd.conf.in.

protocols plugin: Add a plugin which collets protocol information.

Currently a Linux only plugin. This plugin collects information about
IP, TCP, UDP, etc.

Thanks to Ulf Zimmermann for his (indirect) suggestion ;)

Merge branch 'ff/java'

Merge branch 'collectd-4.6'

Merge branch 'collectd-4.5' into collectd-4.6

dns plugin: Don't pass a NULL pointer to `pcap_open_live'.

Although the documentation states clearly, that passing a NULL pointer
as device is okay and handled like "any", doing so will crash the daemon
on some systems, most notably *BSDs but Linux users have reported this
behavior, too.

This patch passes "any" when the pointer it NULL, which reportedly
resulted in a different behavior, but still crashing the daemon. We'll
keep trying ;)

oracle plugin: Fix type size issue under 64bit systems.

Under 64bit systems, `ub4' is of a different size than `size_t', so that a
`(size_t *)' cannot be casted to a `(ub4 *)' automatically.

contrib/oracle/create_schema.ddl: Add GPLv2 license information.

In accordance with Roman Klesel,
<C937BDA691901BFAD72AFEF3@ganga.office.noris.de>

src/common.c: Fixed a race condition in check_create_dir().

Between checking for the existence of a directory using stat() and creating
the directory using mkdir(), another thread might have already created the
directory thus causing mkdir() to fail with errno == EEXIST. This case is now
handled sanely, no longer causing the function (and thus some write callback)
to fail in this case.

Most likely, this only happens during startup when creating the data
directories - later, no two threads should try to create the same directory.

Interestingly enough, I frequently encountered this issue on a single core
machine.

collectd.conf(5), collectd-java(5): Updated the documentation for the Java plugin.

collectd-java(5): Documented the `match' and `target' callbacks.

bindings/java/: CollectdMatchFactoryInterface: Add more JavaDoc.

bindings/java/: Unify inline documentation terms.

s/function/method/

java plugin: Add support for `target' callbacks.

With the ``do everything while in the configuration phase'' issues all
sorted out, this was actually kinda easy.

Matches and targets share a lot of code, so that the patch actually isn't
all that large..

Now, the Java plugin is feature-complete \o/

Merge branch 'master' into ff/java

Merge branch 'collectd-4.6'

src/plugin.c: Assure that targets get dynamically allocated memory.

If targets want to replace the values, they will have to use dynamically
allocated memory. If they can't free the values, because the pointer
might point to statically allocated memory, memory will be lost.

Unfortunately stack allocation will not do, since we will then not be able
to detect multiple replacements.

To impose as little a performance issue as possible, the dynamic allo-
cation is only done when either chain is present. If the filter mecha-
nism is not used, the values will not be copied.

java plugin: Add support for `match' callbacks.

Holy crap, that one needed some serious magic.. The problem is, that the
filter chains are created before the initialization functions are run.
Since the Java plugin used to initialize the JVM and load the classes in
the init function, the match callbacks were not available in time.

The behavior is now: Create the JVM as soon as the the first `LoadPlugin'
option is found and load and configure all Java plugins while in the
configuration phase. (I. e. exactly like C plugins.)

Phew, I've earned that beer now.

logfile plugin: Interpret `(notification_t *)->time == 0' as `now'.

java plugin: Expose `plugin_dispatch_notification' to Java plugins.

src/collectd.conf.in: Add `LoadPlugin java'.

src/plugin.c: Fix plugin_dispatch_notification again.

java plugin: Add support for `notification' callbacks.

src/plugin.c: Add a user_data_t pointer to notification callbacks.

bindings/java/: Add some javadoc markup to the interfaces.

collectd-java(5): Fix a typo.

Updated various copyright information.

collectd.conf(5): Updated the information on the java plugin.

java plugin: Add support for `log' callbacks.

src/plugin.c: Add a user_data_t pointer to log callbacks.

Updated various copyright information.

java plugin: Add support for `flush' callbacks.

src/plugin.c: Add a user_data_t pointer to flush callbacks.

collectd-unixsock(5): Slightly improved the description.

Merge branch 'master' into ff/java

Merge branch 'collectd-4.6'

Conflicts:

src/bind.c

Bumped version to 4.6.1; Updated ChangeLog.

Merge branch 'collectd-4.5' into collectd-4.6

Conflicts:

ChangeLog
version-gen.sh

The changes made in b6612126 and reverted in 5fa56b89 are retained in
collectd-4.6.

Revert "libiptc/Makefile.am: Added xtables.h, libxtc.h to libiptc_la_SOURCES."

This reverts commit b6612126d862d182d9219e0512fa9ca833272b62.

This commit should have gone in the master branch, not in the
collectd-4.5 branch.

Bumped version to 4.5.3; Updated ChangeLog.

java plugin: Rename `CollectdAPI' to `Collectd'.

collectd-java(5): Update the documentation.

Especially add the new ``native'' methods.

java plugin: Change the name of all functions to conform to the Java convention.

Revert part of 5f195369: Accidentally commited changes to src/Makefile.am.

java plugin: Expose `plugin_log' to Java plugins.

java plugin: Added comments for most functions.

Hopefully this'll make the code a bit easier to read.

java plugin: Implement registration for the other methods.

``Other'' meaning `Config', `Init', and `Shutdown' right now.

collectd-java(5): Updated the documentation.

java plugin: Implemented write plugin registration.

java plugin: Improve the read registration code.

java plugin: Implement plugin registration via `RegisterRead'.

src/plugin.c: Remove all read functions before calling the shutdown functions.

This way the user_data may be cleaned up before the final cleanup
phase..

java plugin: Change the API to use the `DataSet' class.

java plugin: Use the changed plugin infrastructure to call read and write functions directly.

The read and write functions implemented in Java are now registered with
the "complex" interface as "java:<class name>". This way the cjni_read and
cjni_write functions can determine which Java function to call.

The patch is bigger than it'd need to be, because the order of some
functions has been changed..

src/plugin.c: Change the write callbacks to receive a user_data_t pointer.

There are by far not as many write callbacks, so I've just changed the
callback in general rather than introducing a `complex write' callback.

src/plugin.c: Implement `plugin_register_complex_read'

These read callbacks will receive a user pointer. This is nice for the
Java plugin, because the plugin infrastructure can now call one specific
Java read function.

AfaIk, the Perl plugin can make use of this, too.

java plugin: Update the full name of the moved classes.

Revert "src/plugin.[ch]: Implement `plugin_dispatch_values_async'."

This reverts commit ba6497bdffb6a225eb6c86c8b8fa57a2520f0c5f.

bindings/java: Moved the `PluginData' and `ValueList' classes.

And implemented various (trivial) `Collectd*Interface' classes.

java plugin: Implement a reference counter for the JVMEnv pointers.

This way one thread entering the Java plugin twice, e. g. with the
following call-chain, will not detach itself from the JVM before it is
completely done with it.

The problematic situation is:
  -> cjni_read
  -> ALLOC JVM
  -> `Read' (in Java)
  -> `DispatchValues' (in Java)
  -> plugin_dispatch_values
  -> cjni_write
  -> ALLOC JVM (this is a no-op!)
  -> `Write' (in Java)
  -> DEALLOC JVM

This last dealloc is prematurely, because the thread is not done with
the JVM yet: It'd like to continue and return from `DispatchValues' to
execute some more Java code..

types.db(5): Added a note about distributing custom types.db files.

perl plugin: Deprecated the use of plugin_register() to register data-sets.

A custom types.db file should be used instead.

Merge branch 'ff/java' of /var/lib/git/collectd into ff/java

collectd-java(5): Fix a typo.

bind plugin: Fix a typo.

java plugin: Use the new `plugin_dispatch_values_async' function.

Having Java read- and write-functions at the same time works now \o/

src/plugin.[ch]: Implement `plugin_dispatch_values_async'.

This function differs from `plugin_dispatch_values' in that it will add
the value_list_t to a queue rather than calling the write functions
right away. This as at least two advantages:

  - The _async function will often return faster, since no file
    operation is done.

  - The ``read thread'' and the ``write thread'' are not the same
    thread. This makes it much easier for plugins that provide both,
    `read' and `write' callbacks.