Merge pull request #3096 from rubenk/remove-errors-option-from-pod2man

pod2man: remove --errors argument

Merge branch 'collectd-5.8'

pod2man: remove --errors argument

Older versions of pod2man don't have this

Merge branch 'collectd-5.7' into collectd-5.8

Build system: enable the "df" plugin when getmntent_r() is available.

Looks like the check result used to be stored in a variable of a
different name.

Auto-Merge pull request #3065 from btoneill/rrdcached

Automatically merged due to "Automerge" label

rrdcached plugin: Inline time conversion.

rrdcached plugin: fixed issue with typing

rrdcached plugin: changed time format to no longer lose microseconds when sending to rrdcached.

See https://lists.oetiker.ch/pipermail/rrd-developers/2003-March/001007.html for we want this

Merge pull request #3060 from octo/issue/2973

contrib/docker/: Switch to Debian slim, separate ENTRYPOINT/CMD, install suggestions.

Auto-Merge pull request #3090 from rubenk/lua-plugin-prevent-leak-on-shutdown

Automatically merged due to "Automerge" label

Merge pull request #3074 from mfournier/ci-setup

Re-implement CI using cirrus-ci

Format src/lua.c.

Merge pull request #3092 from rubenk/pod2man-dont-hide-errors

Don't hide errors when creating manpage

Merge pull request #3093 from rubenk/sensu-fix-warning

Sensu fix warning

sensu plugin: fix compiler warning when build with --enable-debug

In file included from src/write_sensu.c:31:
src/write_sensu.c: In function ‘sensu_config’:
./src/daemon/plugin.h:398:20: warning: ‘%s’ directive argument is null [-Wformat-overflow=]
  398 | #define DEBUG(...) plugin_log(LOG_DEBUG, __VA_ARGS__)
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/write_sensu.c:1227:7: note: in expansion of macro ‘DEBUG’
 1227 |       DEBUG("write_sensu plugin: Got tag: %s", tmp);
      |       ^~~~~
src/write_sensu.c:1227:43: note: format string is defined here
 1227 |       DEBUG("write_sensu plugin: Got tag: %s", tmp);
      |

Don't hide errors when creating manpage

Apparently the default behaviour for pod2man changed at some point and
now it exits with a syntax error instead of exiting successfully with a
<POD ERRORS> section.

Now we do display the syntax issue:
[ruben@localhost collectd]$ make -j
  POD2MAN  src/collectd.conf.5
src/collectd.conf.pod around line 10361: You forgot a '=back' before '=head2'
POD document had syntax errors at /usr/bin/pod2man line 69.
make: *** [Makefile:9641: src/collectd.conf.5] Error 255

Merge pull request #3091 from rubenk/reduce-allocations

Reduce the nr of allocations when parsing types.db

Please clang

Reduce the nr of allocations when parsing types.db

We allocate a number of times for each line in types.db.
Since plugin_register_dataset() makes a copy of the dataset anyway, we
can create the dataset on the stack instead of the heap.

This reduces the number of calloc calls on my system from 624 to 337
when running collectd -T with the default types.db.

lua plugin: cleanup properly when shutting down

Found while running collectd -T under valgrind:

==25971== 75 (64 direct, 11 indirect) bytes in 1 blocks are definitely lost in loss record 154 of 274
==25971==    at 0x483AB1A: calloc (vg_replace_malloc.c:762)
==25971==    by 0x4854009: lua_cb_register_read (lua.c:288)
==25971==    by 0x4FBE886: ???
==25971==    by 0x4FD4900: ???
==25971==    by 0x4FBEAD7: ???
==25971==    by 0x4FBEB04: ???
==25971==    by 0x4FBDF16: ???
==25971==    by 0x4FBEE5E: ???
==25971==    by 0x4FB53ED: ???
==25971==    by 0x48536E7: lua_script_load (lua.c:465)
==25971==    by 0x485387E: lua_config_script (lua.c:532)
==25971==    by 0x485395E: lua_config (lua.c:557)

Merge pull request #3085 from rubenk/handle_simple_config_errors

Handle failure of simple config callbacks

Merge pull request #3086 from rubenk/reduce-configure-noise

Reduce the amount of output from ./configure

Merge pull request #3088 from rubenk/fix-syntax-in-collectd-lua-manpage

Fix syntax in collectd lua manpage

Fix some rendering issues in collectd-lua(5)

collectd.conf(5): fix a typo

Reduce the amount of output from ./configure

All these flags can be found in config.log if needed.

Merge pull request #3080 from rubenk/rrdcached-cleanup

Rrdcached cleanup

Clarify error message a bit

It's not really the reading of the file(s) that fails, they can be read
fine, there are just errors in the config.

Do not suggest to read the logs if parsing config fails

There are no logs at this point, all output goes to stderr.

Furthermore, spreading a log message over two lines is bad practice.

Handle failure of simple config callbacks

When they return an error, collectd should not ignore this.
Fixes #3076

rrdcached: a few minor cleanups

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

cirrus.yml: add --disable-dependency-tracking option by default

cirrus.yml: fix a couple of typos

rrdcached plugin: use designated structure initializer

rrdcached plugin: use same attribute name for both prototypes

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.

Re-implement CI using cirrus-ci

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.

contrib/docker/: Separate CMD/ENTRYPOINT.

This allows users to pass different flags to collectd.

contrib/docker/: Switch to Debian slim.

contrib/docker/: Install suggestions.

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.