Merge branch 'collectd-5.2' into collectd-5.3

src/configfile.c: Let errors in included files propagate up to cf_read().

cf_read_generic() returned NULL in two cases: Unable to read file and
file was empty. Since we wanted to allow empty includes, the "include
file with errors" case was not handled properly.

Github: #323

contrib/redhat/init.d-collectd: Rename variable to …PIDDIR.

Github: #298

allow for non-root user to run script.
As collectdmon handles its pidfile, this would require to change
COLLECTMONPIDIR to something else than only-root writeable /var/run

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

Merge branch 'collectd-4.10' into collectd-5.2

Don't notify continuously when MySQL slave SQL thread is running

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

Add "=encoding" header to all manual pages.

http://bugs.debian.org/708718

Merge branch 'yaccz/curl-status'

curl plugin: Use cdtime to calculate response times.

curl plugin: Don't abort when reading the response code failed.

curl plugin: Remove the curl-only auxiliary configuration functions.

curl plugin: Fix compile issue (long vs. long*).

exec plugin: Document that defaulting to PUTVAL is no longer done

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

Merge branch 'more-varnish-stats'

Merge branch 'document-varnish-instance-block' into collectd-5.2

modbus plugin: Refactor the new reconnection code.

Avoid the goto() and switch() statements.

Github: #294

Cleanup modbus connection if wrong result

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

Fix modbus segfault / New strategy for connecting

The previous implementation had a reconnection loop which should not
really be necessary. Idea tried here is to check if connection is alive,
and establish it if not. Change was begun because previously connection
would only be attempted as a retry, so on first pass a NULL was handed to
libmodbus which caused segmentation faults reproducibly.

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

src/configfile.c: Add the "AutoLoadPlugin" option.

When enabled, <Plugin ...> blocks will automatically load plugins.
Thanks to Tim Bunce for suggesting this!

Github: #333

src/plugin.c: Fix a comment regarding duplicate plugin_load() calls.

src/plugin.c: Move the perl and python "global" magic to plugin_load().

The config file handling is really not the best place to have this. Also,
the feature I'm working on would have to copy this magic if left in the
config code, and I *really* don't want that.

collectd.conf(5): Document the new "LoadPlugin" behavior.

Github: #333

src/plugin.c: Make plugin_load() idempotent, i.e. don't load plugins twice.

Previously, multiple 'LoadPlugin "foo"' lines would dlopen the same .so file
twice and call its module_register function twice. This would eventually result
in complaints about callbacks with the same name being registered. With this
patch each plugin is only loaded only once, allowing people to write "self
contained" configuration segments.

Thanks to Tim Bunce for suggesting this!

Github: #333

Merge branch 'ef/mic'

README: Document power consumption being collected by the mic plugin.

mic plugin: Fix indentation issue.

mic plugin: Change the name of the power settings …

… to reflec the names documented in the manpage.

varnish plugin: update config docs for new collectors

varnish: hint about missing <Instance> block

The warning when the Collect* configuration options where used outside
of an <Instance> block was confusing. This patch warns the user the same
way the apache plugin does.

Improve varnish section in manpage.

There previously was no mention of the mandatory <Instance> block.

Drop suffixes, now that we are putting them in correct types

Do proper storage of power numbers, do conversion to base units.

Add Documentaion for Power collection in the mic plugin

mic plugin: Code for power readings.

Annotate meta data for db plugins

Provide a MetadataFrom setting to name columns whose content is to be
added as metadata to the data collected.  Similar in usage to the
ValuesFrom setting.

add MeasureResponseCode refs #319

curl_xml plugin: Allow arbitrarily long callback names.

Fixes issue #286.

src/common.[ch]: Add the ssnprintf_alloc() function.

src/plugin.c: Allocate read-callback names on the heap.

This allows to use arbitrarily long callback names.
Prerequisite for fixing #286.

collectd.conf(5): Add documentation for the MIC plugin.

configure.in: Remove the unused "plugin_mic" variable.

mic plugin: Use the "core number" as plugin instance.

Previously, per-core CPU statistics would be dispatched as (e.g.):

  <host>/mic-0/cpu-1-user

This makes graping hard, because it required to parse the type instance to
seperate out different graphs. The core number has therefore been moved to the
plugin instance, so the "mic" plugin behaves more like the "cpu" plugin, e.g.:

  <host>/mic-0-cpu-1/cpu-user

mic plugin: Some more coding style changes. Mostly breaking long lines.

mic plugin: Remove the NUM_THERMS define.

Also some coding style changes: Limit line length, move temporary variables
to an innter scope, print the temperature name in an error message (rather
than its ID).

AUTHORS, README: Add Evan and the mic plugin.

cleanup mic number initializing. Prefix Debug and error messages

Use proper values out of memory utilization

Fix input parameter

Cleanup errors from upstream review, cleanup typeographical errors.

here's some bits to fix CFLAGS to CPPFLAGS

rework dependencies for mic plugin

fix copy framework error

AUTHORS, README: Add the "cgroups" plugin.

cgroups plugin: Rename the "cgroups_cpuacct" plugin.

The new name is sufficient and much easier to type / remember.

varnish plugin: add missing fields

This commit adds a large number of fields exported by the varnish stats
interface which have not previously been collected. A number are still missing,
as it is not immediately obvious which stat group they should go in:

    accept_fail
    client_drop_late

and, from Varnish 3.X:

    vmods
    n_gzip
    n_gunzip

In addition, there are a number of fields which were added at the end of the 2.X
series, but in the absence of a way of detecting whether we're compiling against
2.0.X or 2.1.X, are necessarily disabled when compiling against 2.X:

    dir_dns_lookups
    dir_dns_failed
    dir_dns_hit
    dir_dns_cache_full
    backend_retry
    fetch_1xx
    fetch_204
    fetch_304

collectd.conf(5): Fixed documentation for include filter patterns.

The syntax has been changed from a positional argument to using an <Include>
block with additional arguments.

Drop st_mysql_options and use db->con->options instead

Included memory usage per VM

Included all the metrics about the actual memory used by VMs (as
returned by libvirt.virDomainMemoryStats)

collectd.conf(5): Fixed a typo (comma vs. dot) and markup fixes.

src/types.db: Remove the unused "cpuacct" type.

cgroups_cpuacct plugin: Read file line-by-line.

Use the field name from the file as type instance when dispatching values.

dns plugin: Sleep for an interval before re-trying to read from interface.

Otherwise we'd just have the CPU hogging behavior shifted from libpcap
to collectd …

Github: #308

varnish plugin: missing definition in default config

dns plugin: Resume after receiving a PCAP_ERROR from pcap_loop().

If an interface goes down while capturing, libpcap returns PCAP_ERROR.
Handle this case gracefully.

Github: #308

Add Alias and ConnectTimeout options to MySQL plugin

write_riemann plugin: Really fix a memory leak.

Github: #307

Add config options

write_riemann plugin: Fix a memory leak.

Attributes were allocated and added to the event but not freed. D'oh!
Thanks to @dch for reporting this!

Github: #307

add cpu stat gathering

Add Thermal gathering, and the config properly

Initial implementation of build addition, and memory gathering for the mic card

cgroups_cpuacct plugin: Re-indent read_cpuacct_root().

The rest of the file is indented using tabs.

cgroups_cpuacct plugin: Use type "cpu" to submit value lists.

cgroups_cpuacct plugin: Minor style fixes.

* Avoid mixed declarations.
* Use parse_value() rather than atoll().
* Comment static arguments to walk_directory().
* Return an error if the read() function can't locate the cgroup
  mount point.

src/common.[ch]: read_file_contents: Use {s,}size_t rather than int.

Add cgroups_cpuacct plugin to collect CPU user/system time per cgroup

This is handy for computers using systemd, as systemd puts each service
into a separate cgroup.

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

src/utils_mount.h: Add <stdio.h>.

This is required on Solaris 10. Thanks to dannypoo@ for reporting this
problem.

Github: #306

Merge branch 'cm/lvm'

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

AUTHORS, README: Add Chad and the LVM plugin.

Changed plugin from volume to lvm

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

Removed un-needed files

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

Record load averages over 100

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

amqp: add Graphite{SeparateInstances,AlwaysAppendDS} options

volume plugin: Fix a couple of issues from the code review.

* Use df_complex as data type.
* Submit LV sizes and "free", but not "total" and "used".
* Fix error handling; don't use variable if it is NULL, remove calls to
  exit().

snmp plugin: Removed "mixed delaration".

This should fix this warning Fabien Wernli reported:
snmp.c:1381: warning: ISO C90 forbids mixed declarations and code

Changed data type from vg to vol_group and lv to logical_vol to be more descriptive

changed symbol check for lvm2app

added missing comma to AC_CHECK_LIB for lvm2app

Added logical volume size metrics

src/Makefile.am: Really don't distribute *.pb-c.[ch].

src/Makefile.am: Add *.proto to EXTRA_DIST. D'oh!

Build system: Don't ship the generated *.pb-c.[ch] files.

They depend on specific versions of the protobuf-c.h header file, so
shipping this will cause problems for people having a different version
of protoc-c installed. This reverts some of the work done in #269, but
distcheck should still work. Meh.

RedHat RPM spec: updated for 5.3.0

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

configure.in: Fix libgcrypt flags on FreeBSD.

On FreeBSD, AM_PATH_LIBGCRYPT sets LIBGCRYPT_CFLAGS but not
LIBGCRYPT_CPPFLAGS. Use both variables to be on the save side.

Bump version to 5.3.0; Update ChangeLog.

Merge branch 'collectd-5.2'

Bump version to 5.2.2; Update ChangeLog.

Merge branch 'collectd-5.1' into collectd-5.2

Conflicts:
ChangeLog
version-gen.sh

Bump version to 5.1.3; Update ChangeLog.

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

Conflicts:
ChangeLog
version-gen.sh