Merge pull request #3078 from nward/document_snmp_complex_reads

Document the snmp plugin's use of complex reads

Update documentation to reflect how threads really work these days.
Tidy up a few end of line whitespaces (editor does it automatically).

Merge pull request #3070 from rubenk/travis-improvements

Travis improvements

Travis: try to fix env settings

Travis: try to fix protobuf header detection

Travis: try to make mysql plugin build on OSX

Merge pull request #3061 from octo/docs

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

Travis: Use two cores to run make

Travis instances have two cores, see https://docs.travis-ci.com/user/reference/overview/#virtualization-environments

Travis: fetch just the tip of the git branch

Travis: we don't use submodules

Travis: silence git clone

Travis: install dependency for amqp1 plugin on OSX

Travis: explicitly set compiler to clang on OSX

Travis: build collectd with debug enabled

Travis: only use clang on OSX

gcc is an alias for clang

Travis: install the right dependency on Xenial

libmosquitto0-dev became libmosquitto-dev

Travis: use newer Xcode image

Travis: do a verbose autoreconf

Travis: disable dependency tracking

Should speed up the build a bit

Travis: switch to Xenial

Trusty is almost end of life

Travis: remove obsolete option

The container builds are deprecated:
https://changelog.travis-ci.com/the-container-based-build-environment-is-fully-deprecated-84517

Run make distcheck instead of make && make check

The advantage being that this tests that all needed files end up in the
tarball and that make clean leaves nothing behind

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.