Merge branch 'collectd-5.8'

Conflicts:
src/sensors.c

Merge branch 'collectd-5.7' into collectd-5.8

Auto-Merge pull request #3063 from octo/issue/3062

Automatically merged due to "Automerge" label

strjoin(): Fix behavior if output buffer is NULL.

Additionally, test the (NULL, 0) output buffer for all test cases rather
than an individual, isolated test case.

Fixes: #3062

Auto-Merge pull request #3058 from cjmayo/debug_utf8

Automatically merged due to "Automerge" label

Merge pull request #3057 from cjmayo/style_smart_temperature

collection3: Add Type for smart_temperature

Merge pull request #3059 from octo/sock_cloexec

write_prometheus plugin: Only use SOCK_CLOEXEC if it is defined.

collection3: Output graph.cgi debug as UTF-8

Ensure text and symbols e.g. degrees Celsius are shown as intended.

collection3: Add Type for smart_temperature

Copied from Type temperature.

write_prometheus plugin: Only use SOCK_CLOEXEC if it is defined.

Fixes compilation issue on Mac OS X. See also #3055.

Merge pull request #3053 from collectd/code_owners

CODEOWNERS: Add code owners as discussed at the meetup.

Merge pull request #3054 from octo/docs

CONTRIBUTING.md: Document the new change log requirement / behavior.

CONTRIBUTING.md: Document the new change log requirement / behavior.

CODEOWNERS: Add code owners as discussed at the meetup.

See also: https://help.github.com/articles/about-code-owners/

Merge pull request #3047 from cjmayo/style_border

collection3: Improve index.cgi form fieldset border style

collection3: Improve index.cgi form fieldset border style

- Fit to the form contents instead of filling the page width
- Relate better to the table below by aligning the left-hand side and
  increasing the space between

Merge pull request #3040 from AlexZzz/collectd-5.8

Backport: netlink plugin: Use of less strict rules in link_filter_cb()

Merge pull request #3042 from srivasta/upstream-fixes

[python-3.7-fix]: Allow compilation with Python 3.7

[python-3.7-fix]: Allow compilation with Python 3.7

PyOS_AfterFork() is deprecated in favour of the new functions
PyOS_BeforeFork(), PyOS_AfterFork_Parent() and
PyOS_AfterFork_Child(). (Contributed by Antoine Pitrou in bpo-16500.)

Since the -Werror=deprecated-declarations flag is on, the fact that
PyOS_AfterFork is deprecated now results in a warning, and since all
warnings being treated as errors, collectd fails to compile.

This commit detects when we have python 3.7 or newer, and uses the new
replacement function.

Signed-off-by: Manoj Srivastava <srivasta@golden-gryphon.com>

netlink plugin: Use of less strict rules in link_filter_cb()

mnl_attr_validate2() function implements strict equality check of kernel and
userspace structures size. Additional counters was added to 4.6 Linux kernel,
sizes was changed and mismatch can occur.

This patch weakened validation.
Now Collectd just checks if structures, received from kernel space, has enough
data.

Closes: #2510

# Conflicts:
# src/netlink.c

processes: Reduce procstat entry 'age' limit

The 'age' value can grow only when process terminated.

There is no sence to wait 10 read cycles for process rebirth.

Theoretically, this reduces probability for process data takeover by
another process with same pid.

Issue: #3026

processes: Fix 'delay_rate' race condition

When ts_delay_by_tgid() failed (for example, when observable process
terminates), `ps->delay` fields are not updated and remain set to zero.

This interpreted as counter overflow and produces spike.

Issue: #3026

collectd.conf(5): finish a sentence

collectd.conf(5): fix a few typos

Rewrite configure check for CUDA

It had a bunch of issues, one of them that it touched CFLAGS, the other
that it didn't use CUDA_CFLAGS.

The check is still too simple, we need to check if we can actually use
the library instead of just the header.

Treewide: do NUL-termination correctly

This is purely cosmetic but still, it sets a good example

Fix build

lookup_identifier_t has a few array members.
To zero-initialize these we need a pair of extra braces.

See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80454

Fix a couple of warnings in test suite

src/utils_vl_lookup_test.c: In function ‘checked_lookup_add’:
src/utils_vl_lookup_test.c:91:3: warning: ‘strncpy’ specified bound 128 equals destination size [-Wstringop-truncation]
   strncpy(ident.host, host, sizeof(ident.host));
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/utils_vl_lookup_test.c:92:3: warning: ‘strncpy’ specified bound 128 equals destination size [-Wstringop-truncation]
   strncpy(ident.plugin, plugin, sizeof(ident.plugin));
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/utils_vl_lookup_test.c:93:3: warning: ‘strncpy’ specified bound 128 equals destination size [-Wstringop-truncation]
   strncpy(ident.plugin_instance, plugin_instance,
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           sizeof(ident.plugin_instance));
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/utils_vl_lookup_test.c:95:3: warning: ‘strncpy’ specified bound 128 equals destination size [-Wstringop-truncation]
   strncpy(ident.type, type, sizeof(ident.type));
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/utils_vl_lookup_test.c:96:3: warning: ‘strncpy’ specified bound 128 equals destination size [-Wstringop-truncation]
   strncpy(ident.type_instance, type_instance, sizeof(ident.type_instance));
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/utils_vl_lookup_test.c: In function ‘checked_lookup_search’:
src/utils_vl_lookup_test.c:113:3: warning: ‘strncpy’ specified bound 128 equals destination size [-Wstringop-truncation]
   strncpy(vl.host, host, sizeof(vl.host));
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/utils_vl_lookup_test.c:114:3: warning: ‘strncpy’ specified bound 128 equals destination size [-Wstringop-truncation]
   strncpy(vl.plugin, plugin, sizeof(vl.plugin));
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/utils_vl_lookup_test.c:115:3: warning: ‘strncpy’ specified bound 128 equals destination size [-Wstringop-truncation]
   strncpy(vl.plugin_instance, plugin_instance, sizeof(vl.plugin_instance));
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/utils_vl_lookup_test.c:116:3: warning: ‘strncpy’ specified bound 128 equals destination size [-Wstringop-truncation]
   strncpy(vl.type, type, sizeof(vl.type));
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/utils_vl_lookup_test.c:117:3: warning: ‘strncpy’ specified bound 128 equals destination size [-Wstringop-truncation]
   strncpy(vl.type_instance, type_instance, sizeof(vl.type_instance));
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  CCLD     test_utils_vl_lookup

Ignore test-suite.log in all directories

Merge pull request #3022 from rubenk/treewide-fix-some-style-issues

Treewide: fix some style issues

xencpu plugin: fix minor style issues

virt plugin: fix some minor style issues

utils_rrdcreate.c: fix minor style issue

utils_ovs.c: fix some minor stylistic issues

utils_db_query.c: fix some minor style issues

snmp_agent plugin: fix minor nit

postgresql plugin: fix some stylistic issues

ovs_stats plugin: fix some stylistic issues

oracle plugin: remove unneccesary cast

liboconfig: fix minor style issues

libcollectdclient: fix minor style issue

java plugin: fix minor style issue

ipmi plugin: remove unneccesary cast

intel_rdt plugin: fix some minor style issues

intel_pmu plugin: fix some minor style issues

gmond plugin: fix minor style issues

disk plugin: fix minor style issue

daemon/utils_cache.c: fix minor style issue

daemon/types_list.c: fix minor style issue

daemon/plugin.c: fix two more stylistic issues

daemon/plugin.c: remove unneccesary casts

daemon/plugin.c: fix minor style issue

cpufreq plugin: fix minor style issue

barometer plugin: remove unneccesary cast

collectd-nagios: remove unneccesary casts

ceph plugin: use recommended style for calloc

Merge pull request #3017 from rpv-tomsk/collectd-collectd-5.8

configure.ac: run dpdk build tests only if pkgconfig fails

configure.ac: run dpdk build tests only if pkgconfig fails

The AC_CHECK_LIB test runs unconditionally, and fails with DPDK 18.11
when built with Meson as there is no longer a libdpdk.so linker script,
but only a pkg-config file, so -ldpdk (which is what AC_CHECK_LIB runs)
fails.

Use AC_LINK_IFELSE instead, with compiler and linker flags set
appropriately.

Signed-off-by: Luca Boccassi <bluca@debian.org>

Merge pull request #3015 from bluca/dpdk_pkgconfig

configure.ac: run dpdk build tests only if pkgconfig fails

configure.ac: run dpdk build tests only if pkgconfig fails

The AC_CHECK_LIB test runs unconditionally, and fails with DPDK 18.11
when built with Meson as there is no longer a libdpdk.so linker script,
but only a pkg-config file, so -ldpdk (which is what AC_CHECK_LIB runs)
fails.

Use AC_LINK_IFELSE instead, with compiler and linker flags set
appropriately.

Signed-off-by: Luca Boccassi <bluca@debian.org>

virt plugin: Add missing config_keys

virt plugin: New options: ReportBlockDevices and ReportNetworkInterfaces

This allows to disable unneeded reports if stats are gathered by another plugin.

virt plugin: Move adding of devices into own functions

This also covers case when error while adding block device
refuse adding of domain network devices.

virt plugin: Fixed mistake in variable rename

virt plugin: Handle ignorelists properly

Added check for ignorelist_create() success.
Removed unnecessary "not null" checks in rest of code.

virt plugin: Updated InterfaceFormat documentation

Merge pull request #3014 from rpv-tomsk/collectd-master

sensors plugin: Drop support for libsensors older than 3.0.0

Merge pull request #3000 from teknoraver/master

Add ovs-dpdk extended metrics

indent with contrib/format.sh

Add ovs-dpdk extended metrics

Signed-off-by: Matteo Croce <mcroce@redhat.com>
Signed-off-by: Ryan McCabe <rmccabe@redhat.com>

Merge pull request #3013 from rpv-tomsk/collectd-collectd-5.8

sensors: Removed checks for upper limit of SENSORS_API_VERSION

sensors plugin: Drop support for libsensors older than 3.0.0

The lm_sensors-3.0.0, which introduces 'modern' API, released 2007-11-24.
The time came to drop older version support.

sensors: Removed checks for upper limit of SENSORS_API_VERSION

That makes no more sense after lm-sensors got new maintainers.

Issue: #3006

sensors: Removed checks for upper limit of SENSORS_API_VERSION

That makes no more sense after lm-sensors got new maintainers.

Closes: #3006

Merge pull request #3012 from rpv-tomsk/collectd-master

ovs_stats: Cleanup and improve

ovs_stats: Optimize read callback

ovs_stats: Do not create bridge if it has not be monitored.

ovs_stats: Fix indent

ovs_stats: Locking reworked

Locking now done in unified way, on top-level.
That removes usage of global variables in unprotected way.

ovs_stats: Do not create ports if there is no bridge name

That has no sence, as it will not pass condition in ovs_stats_plugin_read()

Merge pull request #3011 from rpv-tomsk/collectd-master

ovs_stats: Code style and cleanup

ovs_stats: Removed unused variable

ovs_stats: Fix indents

Merge pull request #2880 from abays/add_ovs_stats_bonds_3

ovs_stats bonds support with separate interface stats

Merge pull request #3010 from rpv-tomsk/collectd-5.8

write_mongodb: Fix plugin dependencies

write_mongodb: Fix plugin dependencies

Issue: #2900

write_mongodb: Fix plugin dependencies

Issue: #2900

Fix merge issue

Merge pull request #2807 from sileht/master

virt: allow read Hostname from libvirt metadata

virt: Fix indent

virt: Merge changes from master to resolve conflict

Merge pull request #2990 from rpv-tomsk/collectd-master

virt plugin: Document Instances option

virt: allow read Hostname from libvirt metadata

This change allows to read Hostname from Libvirt metadata API.

Applications like OVirt or Openstack Nova uses this metadata API to
store additional informations like the 'user facing' Guest name.

To do so, a new choice 'metadata' is added for 'HostnameFormat' and
'PluginInstanceFormat'. And two new options are also added to localize
the hostname into the Guest metadata: HostnameMetadataNS and
HostnameMetadataXPath.

Closes: #2805

Small snprintf optimization

Merge branch 'master' into add_ovs_stats_bonds_3

Document InterfaceStats option for ovs_stats plugin

Merge pull request #2994 from octo/tail-doc

collectd.conf(5): Improve the tail plugin's documentation.

collectd.conf(5): Improve the tail plugin's documentation.

virt plugin: Document Instances option

Documented implementation from 452324c4a5b6ec07575cfc5d5d339852b040509b

Merge pull request #2988 from octo/ff/struct-init-syntax

log_logstash plugin: Fix non-portable struct initialization with "{}".