rrdcached plugin: use designated structure initializer

rrdcached plugin: use same attribute name for both prototypes

Merge pull request #3061 from octo/docs

CONTRIBUTING.md: Improve wording around ChangeLog; fix example.

Merge pull request #3055 from dfandrich/travis

Add OS X to the Travis build matrix.

Merge pull request #3021 from rubenk/libcollectdclient-fix-stringop-warning

Libcollectdclient fix stringop warning

Merge pull request #2961 from octo/ff/utils

Tree wide: Move utilities and libraries to src/utils/.

.travis.yml: Use the "apt" and "homebrew" add-ons.

This might be more optimized than installing ourselves.

.travis.yml: Remove $(nproc).

nproc is not available on Mac OS X.

Tree wide: Move utilities and libraries to src/utils/.

This is a first step that does not yet include *all* "utils_*.c" files,
but we're getting there.

Add OS X to the Travis build matrix.

Merge branch 'collectd-5.8'

Conflicts:
src/sensors.c

Merge branch 'collectd-5.7' into collectd-5.8

docs/CONTRIBUTING.md: s/test/text/

Move CONTRIBUTING.md to the docs/ directory.

CONTRIBUTING.md: Clarify our requirements for a "ChangeLog:" line.

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

libcollectdclient: fix minor cosmetic issue

String should be NUL-terminated, so compare with the NUL char

libcollectdclient: fix gcc warning

make[1]: Entering directory '/home/ruben/src/collectd'
  CC       src/libcollectdclient/libcollectdclient_la-network_parse.lo
In function ‘parse_string’,
    inlined from ‘parse_identifier’ at src/libcollectdclient/network_parse.c:169:7,
    inlined from ‘network_parse.constprop’ at src/libcollectdclient/network_parse.c:540:11:
src/libcollectdclient/network_parse.c:161:3: warning: ‘strncpy’ specified bound 64 equals destination size [-Wstringop-truncation]
   strncpy(out, in, out_size);
   ^~~~~~~~~~~~~~~~~~~~~~~~~~

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

CONTRIBUTING.md: Improve wording around ChangeLog; fix example.

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