tokkee.org

RedHat specfile: updated for 5.2.0

Also includes a couple of cleanups mentioned in github#186.

Allow for conditional compilation of plugins

To enable/disable a plugin you can toggle the
%define for that plugin at the top of the specfile.
You can also pass the plugin as argument to rpmbuild,
for example:
rpmbuild -ba --without rrdtool
or:
rpmbuild -ba --with rrdcached

Quote arguments

No need to edit collectd.conf anymore.

This was fixed by in commit 25fa4ad

Follow Fedora Packaging Guidelines in java subpackage

Cleanup redhat specfile

Fixed a few rpmlint warnings
Updated (mostly) to Fedora Packaging Guidelines

RedHat RPM spec: update for 5.1 + improvements

  * Enabled all buildable plugins based on libraries available on EL6 +
    EPEL
  * All plugins requiring external libraries are now shipped in seperate
    packages.
  * No longer treat Java plugin as an exception, correctly set
    $JAVA_HOME during the build process + ensure build deps are installed.
  * Dropped per-plugin configuration files, as they tend to diverge from
    upstream defaults.
  * Moved perl stuff to /usr/share/perl5/
  * Don't alter Interval and ReadThreads by default, let the user change
    this himself.
  * Drop collection.cgi from main package, as it's been obsoleted by
    collection3
  * Moved contrib/ to its own package, to avoid cluttering the main
    package with non-essential stuff.
  * Replaced BuildPrereq by BuildRequires

RedHat initscript: small improvements

  * checks configuration before (re)starting, based on debian's
    initscript
  * use /etc/sysconfig instdead of /etc/default
  * include optional $ARGS in arguments passed to collectd.

collectd.conf(5): Add the per-plugin interval to the synopsis.

Fix java detection as unprivileged user.

If you have an inaccessible directory inside
JAVA_HOME, find returns an error and the test fails.

I found this on Fedora, where /usr/lib64/audit is only
readable by root, and my JAVA_HOME was set to /usr

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

Fix building of java binding

Caused by d327beed4d1541d883c4823485b381d1dff79039

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

src/libcollectdclient/Makefile.am: Fix typo.

src/libcollectdclient/Makefile.am: Add "collectd/network.h".

Bump version to 5.2.0; Update ChangeLog.

src/libcollectdclient/network.c: Fix the build on FreeBSD.

On FreeBSD, we have to include <netinet/in.h> to get IN_MULTICAST(). We
don't need to include anything extra, as according to POSIX,
<netinet/in.h> also exposes htonl(). There is no need to include
<arpa/inet.h>.

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

src/libcollectdclient/network_buffer.c: Fix gcrypt build on FreeBSD.

As with src/network.c, we need to ensure that we define
GCRYPT_NO_DEPRECATED on FreeBSD to get rid of the compiler warnings
emitted by the header file.

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

Merge branch 'collectd-5.1'

Merge remote-tracking branch 'origin/master'

Merge branch 'collectd-5.0' into collectd-5.1

Merge branch 'collectd-4.10' into collectd-5.0

fixed paths in example configuration file

These paths previously got expanded to:
  #BaseDir "/usr/var/lib/collectd"
instead of:
  #BaseDir "/var/lib/collectd"

And on systems which put libs in /usr/lib64:
  #PluginDir   "/usr/lib/collectd"
instead of:
  #PluginDir   "/usr/lib64/collectd"

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

lcc_features.h: Relicense under the MIT license.

src/libcollectdclient/network_buffer.c: Copy gcrypt import magic from the network plugin.

libcollectdclient: Update to the high-resolution time format.

Merge branch 'sr/pf'

Merge branch 'sh/postgresql-writer'

Conflicts:
src/collectd.conf.pod
src/postgresql.c

Merge branch 'sh/postgres-queries'

Merge branch 'libvirt-interfacenumber'

GenericJMXConfConnection: Automatically determine the host name.

Fixes Github issue #182.

libcollectdclient: Relicense under the MIT license.

Merge branch 'ff/netlib'

Conflicts:
src/collectdctl.c
src/libcollectdclient/client.c
src/libcollectdclient/collectd/client.h

src/network.c: Fix the build on FreeBSD.

<gcrypt.h> accepts a definition called GCRYPT_NO_DEPRECATED to disable
deprecated functionality. Unfortunately, this definition is not
sufficient to suppress all warnings. However, FreeBSD's version of
libgcrypt has been fixed to properly remove all deprecated features.

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

src/plugin.c: Replace the last use of "interval_g" in this file.

Merge branch 'sh/plugin_interval'

Conflicts:
src/amqp.c
src/exec.c
src/memcached.c
src/unixsock.c

collectd_insert.sql: Use timestamptz rather than timestamp.

Don't want to get messed-up values after daylight-savings changes.

utils_time: cdtime_to_iso8601(): Include timezone information as well.

src/configfile.[ch]: Implement the cf_get_default_interval() function.

This should be able to replace "interval_g" in a global (i.e. non-plugin)
context, such as the interval in which timeouts are being checked.

The default value (10 seconds) is also configurable at compile time using
a define.

postgresql plugin: Simplified transaction handling a bit.

Don't set any next commit time after committing a transaction but only when
starting a new transaction. This way, db->next_commit == 0 always means that
we're outside of a transaction. Also, this removes the need to restart a
transaction right away. Rather, the write callback will take care of that the
next time it gets any data.

collectd.conf(5): Documented the postgresql FLUSH support.

postgresql plugin: Added support for "flushing" data.

This may be used to commit a PostgreSQL writer's transaction. Two different
types of flush callbacks are registered:

 - postgresql: Flush *all* databases.
 - postgresql-<database>: Flush the database named '<database>' only.

postgresql plugin: Fixed unregistering of write callbacks.

Rather than unregistering some callback for each writer for each database,
unregister the single callback for each database only.

postgresql plugin: Unregister all writers on shutdown.

This will make sure that all pending transactions will be committed. Else, the
open transactions would be rolled back by the PostgreSQL backend when closing
the connection.

In order to do so, all database connections are now stored in a plugin-global
array (this will also make further changes possible). Also, a ref-count has
been added to the database object in order to support "deleting" an object
twice when having it in use by a connection doing queries and writes.

postgresql plugin: Commit a transaction before shutting down a connection.

Also, make sure to acquire the database lock before that in order to wait for,
for example, outstanding writers.

postgresql plugin: Don't try to restart a failed transaction when there's none.

postgresql plugin: Restart a writer transaction if the writer's query failed.

The failed query inside the transaction causes the transaction to be aborted
and all subsequent queries to fail as well.

collectd.conf(5): Documented the postgresql <Database> 'Writer' option.

collectd.conf(5): Documented the "CommitInterval" option of 'postgresql'.

collectd_insert.sql: Let values_update_childs() return table names.

The function will now return a set of the names of child tables that have been
created.

collectd_insert.sql: Added view 'collectd'.

This view provides a join of the 'identifiers' and 'values' tables plus an
additional column with the serialized collectd identifier.

postgresql plugin: Added 'CommitInterval' config option.

If specified, this option causes a writer to put several updates into a single
transaction. This transaction will last for the specified amount of time (in
seconds). By default, each update would be executed in a separate transaction
causing quite some overhead.

collectd-perl(5): Don't actually use $interval_g in any examples.

collectd-perl(5): Mention that $interval_g should not be used.

perl plugin: Emit a warning when accessing $interval_g.

Rather, point to plugin_get_interval().

perl plugin: Simplified $interval_g implementation.

Don't pass any pointers to interval_g around just in order to later
dereference them. There's just no reason to do so ;-)

Collectd.pm: Use plugin_get_interval() rather than $interval_g.

Collectd::Plugins::OpenVZ: Use plugin_get_interval() rather than $interval_g.

perl plugin: Export plugin_get_interval() to Perl plugins.

src/plugin.c: Report an error if ctx.interval is not set.

rrdtool plugin: Remove warnings.

They made sense when all data was collected at the same interval. This
has not been true for a while now and these warnings don't make any sense
anymore.

We could transform them into a per-VL check and use complaints to notify
the user, but I don't think it's worth it. There is an "I told you so"
in the manpage in big bold letters ...

src/plugin.c: Don't regularly check for read plugins.

Rather, let plugin_insert_read() wake up all the read threads.
This removes another usage of interval_g.

Merge branch 'collectd-5.1'

Bump version to 5.1.1; Update ChangeLog.

Merge branch 'collectd-5.0' into collectd-5.1

Conflicts:
ChangeLog
version-gen.sh

Bump version to 5.0.5; Update ChangeLog.

Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
ChangeLog
src/pyvalues.c
version-gen.sh

Update version-gen.sh.

Bump version to 4.10.8; Update ChangeLog.

ping plugin: Don't abort the "ping_thread" when ping_send() fails.

This may happen when the network is down. If the thread fails, the read
callback will indicate an error and the exponential back-off will start.
This is not optimal for this scenario, since you usually want to have
ping stats from right when the network is back up.

Fixes Github issue #171.

ping plugin: Refactor "ping_thread": Move iteration over hosts into a function.

disk plugin: Add the "UseBSDName" option for Mac OS X.

This new option will use the "BSD Name" rather than major / minor numbers of
the device when reporting statistics.

See also Github issue #72.

aggregation plugin: Improve two error messages.

Merge branch 'ff/aggregate'

Conflicts:
src/Makefile.am

aggregation plugin: Handle the initial EAGAIN return value of rate_to_value().

This avoids an annoying and confusing warning.

aggregation plugin: Handle a start-up condition gracefully.

The resulting error message would confuse and annoy users.

aggregation plugin: Implement the "GroupBy" option.

This new configuration format has two benefits:
1) It is easier to understand by users, because they don't have to know
   about two types of wildcards.
2) In the future matching of Host, Plugin, Type, ... may support regular
   expressions. In that case, this configuration syntax doesn't need to
   be adapted.

1) Cleans up one configure.ac error AC_LINK_IFELSE requires AC_LANG_PROGRAM (or something similar) to get the setup right.
2) Fix for maintaier-clean so that you can run the autoreconf build script multiple times.
3) Clean up the generated man pages when running cleanup.

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

rrdcached plugin: Fix syntax and type errors.

rrdcached plugin: Implement the "XFF" option.

rrdcached plugin: Implement the "RRATimespan" option.

rrdcached plugin: Implement the "StepSize", "HeartBeat" and "RRARows" options.

rrdcached plugin: Refactor the configuration handling.

Remove a custom string handling function that's only used in one place and use
the functions provided by "configfile.h".

Merge remote-tracking branch 'github-tokkee/sh/empty-config'

collectd-python.conf(5): fix register_* argument description

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

collectd-python.conf(5): change phrasing of function descriptions

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

collectd-python.conf(5): change phrasing of minimum required version

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

collectd-python.conf(5): fix trivial spelling mistakes

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

collectd-python.conf(5): capitalize Python

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

collection3: Remove duplicate definition of contains_invalid_chars().

Thanks to Jonathan Amiez for pointing this out!

Fixes Github issue #160.

pf plugin: Include "common.h".

collectd.conf(5): Documented the postgresql "by_table" queries.

postgresql_default.conf: Added _by_table variants for all table-related qries.

These queries use the 'schemaname' and 'relname' columns to construct the type
instance.

oconfig: Allow empty statement lists (in blocks and files).

This allows to use empty blocks (which is useful during development and
testing) and empty files (which may happen when including config directories,
cf. Debian #592881).

In order not to generate a shift/reduce error, rather than allowing a
'statement_list' to be empty, this has been implemented by explicitly allowing
empty blocks and an empty 'entire_file'.

postgresql plugin: Fixed config key check for <Writer> block.

A missing "else" caused incorrect “Ignoring unknown config key "Statement"”
warning messages.

network.c: call gcry_check_version() as required by library.

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

Merge remote-tracking branch 'trenkel/collectd-4.10' into collectd-4.10

Change the libnetlink function used to query link statistics to match iproute2's behavior and thus not trip on the kernel's new parsing of optional attributes for RTM_GETLINK.

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

postgresql plugin: Added support for using the db instance as query parameter.

collectd.conf(5): Documented the <Database> "Instance" option of postgresql.

postgresql plugin: Added "Instance" option to <Database> config block.

This option may be used to specify the plugin instance that should be used
instead of the database name. This allows to query multiple databases of the
same name on the same host (e.g. when running multiple database server
versions in parallel).

This fixes GH #161 suggested by Bryan Varner.

Merge branch 'collectd-5.1'