Merge branch 'sh/plugin_interval'

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

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.

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.

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'.

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'

Merge branch 'collectd-5.0' into collectd-5.1

Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
src/utils_db_query.c
src/utils_db_query.h

disk plugin: Multiply the average read/write latency with the interval.

The counter/derive will later be divided by the interval, resulting in too
small rates (for interval > 1.0).

Should fix issue #150. Thanks to Manuel Sanmartin for identifying this problem!

Use plugin_thread_create() rather than pthread_create() in all plugins.

This function copies the plugin context from the calling thread to the new
thread. While this might not strictly be necessary in most/many cases, this
will make sure that any code within a plugin uses the same context
information, thus avoiding possible problems in the future.

python: Fix memory leaks.

plugin: Don't switch plugin context in 'write', 'notification' and 'log'.

Rather, keep the context (i.e. interval) information of the calling (read)
plugin. This allows the write, notify and log callbacks to access the correct
interval settings appropriate for the current data-set.

Initialize plugin context to global interval before loading a plugin.

This ensures that the actual interval for each plugin is available through its
context. It is a preparation for removing 'interval_g' (which has limited use
after introducing per-plugin intervals and, thus, its use is rather error-
prone).

Don't use 'interval_g' in any plugins.

Rather, use plugin_get_interval () in order to honor any plugin specific
interval settings.

plugin: Added plugin_get_interval().

This function returns the current value of the plugin's interval. If no
interval has been set in the plugin context, it will fall back to the global
interval or ten seconds as a last resort.

src/utils_format_graphite.[ch]: Add the "store_rates" option.

This functionality must have been lost in the refactoring that added
Graphite support to the AMQP plugin.

network.c: workaround for broken OSX compilers issuing false warnings

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

Merge branch 'collectd-5.1'

Merge remote-tracking branch 'origin/collectd-5.1' into collectd-5.1

Merge branch 'collectd-5.1'

src/utils_db_query.[ch]: Remove duplicate "const" modifier.

Merge branch 'collectd-5.0' into collectd-5.1

Conflicts:
src/snmp.c

Merge branch 'collectd-4.10' into collectd-5.0

Conflicts:
src/snmp.c

Merge branch 'ff/snmp' into collectd-4.10

src/ntpd.c: Write NAN if the peer was not reachable

src/utils_avltree.c: Fix an off-by-one error in c_avl_insert().

When inserting the first node in the tree, the size is not incremented.
This causes c_avl_size() to return one element too little.

Thanks to Yves Mettier for debugging this!

Bugfix (mutex unlock instead of lock)

rc/utils_cache.c: Allocate most of the memory required by uc_get_names() at once.

strdup() will still require additional memory, but no more memory movement like
required when doing realloc() a lot.

This fixes Github issue #148.

collectd.conf(5): Document the "LoadPlugin / Interval" option.

src/plugin.c: Document the use of "register_complex_read".

Also add assertions to make sure plugin_init_ctx() is used correctly.

ntpd plugin: Make some arguments 'const'.

ntpd plugin: Refactor the peer name generation.

Break it out of the main read-loop and put it into two functions: One for the
reference clocks and one for network hosts. The reference clock code will call
the network host code when the clock name is not available in the internal list
of reference clock names. This eliminates the (thread-unsafe) usage of
inet_ntoa(3).

Document ntpd IncludeUnitID and add to default config

src/ntpd.c: Add IncludeUnitID option for backwards compatibility

src/ntpd.c: Format refclock name and unit ID with %s-%u

src/ntpd.c: Add unit number to refclock name

There can be multiple instances of the same refclock with different unit
numbers. The unit number needs to be a part of the peer name, otherwise
the code will try to write the measurements from multiple peers to the
same RRD file.

Merge branch 'ym/utils_cache_bugfix'

src/utils_cache.c: Fix comment re defining LISTVAL_INCREASE.

Using the preprocessor flags (CPPFLAGS) is a tad more correct I think.

snmp plugin: Fix incorrect comparison in csnmp_oid_compare().

the increment is defined as a constant (#define) and will not be a
variable. So its names becomes uppercase.

snmp plugin: Check the return value of csnmp_oid_suffix().

Also fix an issue where the complete OID is compared with only the suffix.

Thanks to Mark Juric to test the changes and point out these problems.

Issue: #131

Add in swap plugin reserved, in and out values for AIX.

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

Merge branch 'mlsr/uptime-aix'

Add libperfstat in Makefile.am

Add AIX support for uptime plugin.

Fix typo in if and return

Fix link with perfstat

AIX suport in contextswitch plugin.

sequence_number is only used in linux.
Generate a compilation error.

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

The commit 3bda88e8b425f43644e303e6957e38ae59c1bc83 change the shell from sh to bash.
To fix the configure script generation error: remove the echo whith \c used in sh.

Note: bash is not installed in AIX by default.
Signed-off-by: Florian Forster <octo@collectd.org>

ARG_MAX on Solaris is *very* large, 2 MB.  Allocating that much on the
stack (local variable) overflows the thread stack and crashes the
process in weird and interesting ways.

"int i" is a useless variable and breaks the build with -Werror.

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

snmp plugin: Fix iterating over tables without an instance configuration.

Also should fix incorrect handling of instance values; the first one would
be skipped.

Performance improvment : uc_get_names now allocates memory by blocks and not element by element.

configure: Only set the Perl interpreter if the argument is a file.

This fixes Github issue #137.

Fixed bugs that prevented to "make distcheck"

Fixes Github issue #132.

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

processes plugin: Coding style and whitespace changes.

processes plugin: Use stack memory when reading the command line.

Also remove the call to the non-threadsafe strtok(3) call and fix the
size passed to sstrncpy().

processes plugin: Call ps_submit_fork_rate() from read_fork_rate().

This avoids the "return ULONG_MAX" hack to signal a problem.