intel_rdt: multiple options are explicitly denied

Add a check to avoid placing multiple Cores or Processes options in
intel_rdt configuration in order to avoid misuse and errors.

Change-Id: I82504a650bf9c568862da9f129a84a3079990496
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>

intel_rdt: Added check if name group is defined before updating of pids list

Change-Id: Ic99acd51be69d2678f25bdde9ca37011ad24fe99

intel_rdt: Fixed compile error in debug build

Change-Id: Ia932f377c5194240aab910d052e422b99ea542a5

utils_proc_pids: moved to utils directory

Change-Id: Ia7c9bf5dca2b871b05c9ed3876059e6d34bd943e

intel_rdt: refactored proc utils to increase efficiency

Change-Id: Ibb0dc6fe59eb77a217ab3b45ec0c1437bc9eb281

intel_rdt: updated pqos mocks for pqos version 3.0

Change-Id: Ibcf8c2fd6d1b490cbeeef7480c49da84389104f4

intel_rdt: close file descriptor on error

Change-Id: I161c8df1e6a4c4c4e0dd4e20601323ef7a1c18f5

Fix for CI error in test code on epel6 platform.

Signed-off-by: Wojciech Andralojc <wojciechx.andralojc@intel.com>
Change-Id: I325ea0728dd1eb65776f239b0d7977cc02f9c8db

intel_rdt: refactor pids monitoring code.
Extract pids config to seperate file.
Extract pids config tests to seperate file.

Change-Id: I2978db56b15a126eced12f40baac631517303e04
Signed-off-by: Mateusz Starzyk <mateuszx.starzyk@intel.com>

intel-rdt: Changed memory allocation of new and lost pids tables from static to dynamic

Change-Id: I3936fe78cfcbb4ce053d38ff1a3236c21eac9fd7
Signed-off-by: Michal Aleksinski <michalx.aleksinski@intel.com>

Corrected issues reported by klockwork

Change-Id: I3737a39c40d89a8782ef9c3d8195351f0fdeb278

intel_rdt: update unit-tests for process monitoring

Refactor certain functions to be testable.

Change-Id: I5adb6ed1b076f0cebc70641f04dd101d4e89b9cc
Signed-off-by: Starzyk, MateuszX <mateuszx.starzyk@intel.com>

intel_rdt: collect data for monitored processes

Change-Id: Ib5a7d318bde69ee8f496b39ca74b3ee4e58d8f5b
Signed-off-by: Starzyk, MateuszX <mateuszx.starzyk@intel.com>

intel_rdt: Backwards compatibility, libpqos v1

Change-Id: I677c8215f89c04cedcb4c9e7d630f187a76a6cc7
Signed-off-by: Andralojc, WojciechX <wojciechx.andralojc@intel.com>

intel_rdt: PID groups change detection

Change-Id: I569d72d0f12dca314e865f0b9236dc09a053c5e5
Signed-off-by: Andralojc, WojciechX <wojciechx.andralojc@intel.com>

intel_rdt: Update config parser for PID monitoring

"Processes" option added.

Change-Id: I7f62a51fc06e0c7503caa7860807f1c8e09b2fa1
Signed-off-by: Andralojc, WojciechX <wojciechx.andralojc@intel.com>

intel_rdt: add PQOS API stubs for pid monitoring.

Add temporary functions to enable further development of rdt plugin.
These stubbed functions will be removed once libpqos extends it's API.

Change-Id: I39dd428e39fd6055774dfc39b4c0690aedee4ebe
Signed-off-by: Starzyk, MateuszX <mateuszx.starzyk@intel.com>

intel_rdt: adds pids extraction for given processes list

adds intel_rdt_tests file with unit-tests of newly introduced functions.

Change-Id: I0e26322caa19f57b7d660807646bd437586cd80d
Signed-off-by: Starzyk, MateuszX <mateuszx.starzyk@intel.com>

Merge pull request #3110 from rubenk/configure-ac-keep-plugin-list-sorted

configure.ac: keep the list of plugins sorted

configure.ac: keep the list of plugins sorted

Merge pull request #3104 from rubenk/intel-rdt-fix-compiler-warnings

intel_rdt plugin: fix compiler warnings

intel_rdt plugin: fix compiler warnings

make[1]: Entering directory '/home/ruben/src/collectd'
  CC       src/intel_rdt_la-intel_rdt.lo
src/intel_rdt.c: In function ‘rdt_dump_cgroups’:
src/intel_rdt.c:74:23: warning: comparison of integer expressions of different signedness: ‘int’ and ‘size_t’ {aka ‘long unsigned int’} [-Wsign-compare]
   74 |     for (int j = 0; j < cgroup->num_cores; j++) {
      |                       ^
src/intel_rdt.c: In function ‘rdt_dump_data’:
src/intel_rdt.c:103:21: warning: comparison of integer expressions of different signedness: ‘int’ and ‘size_t’ {aka ‘long unsigned int’} [-Wsign-compare]
  103 |   for (int i = 0; i < g_rdt->num_groups; i++) {
      |                     ^
  CCLD     intel_rdt.la

Merge pull request #3102 from rubenk/oauth-fix-compiler-warning

oauth.c: fix compiler warning

Merge pull request #3103 from rubenk/write_prometheus-fix-compiler-warning

write_prometheus plugin: fix compiler warning

write_prometheus plugin: fix compiler warning

make[1]: Entering directory '/home/ruben/src/collectd'
  CC       src/write_prometheus_la-write_prometheus.lo
In file included from src/write_prometheus.c:29:
src/write_prometheus.c: In function ‘metric_family_get’:
./src/daemon/plugin.h:393:20: warning: ‘%s’ directive argument is null [-Wformat-overflow=]
  393 | #define ERROR(...) plugin_log(LOG_ERR, __VA_ARGS__)
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/write_prometheus.c:725:5: note: in expansion of macro ‘ERROR’
  725 |     ERROR("write_prometheus plugin: Adding \"%s\" failed.", name);
      |     ^~~~~
src/write_prometheus.c:725:46: note: format string is defined here
  725 |     ERROR("write_prometheus plugin: Adding \"%s\" failed.", name);
      |                                              ^~
  CCLD     write_prometheus.la

oauth.c: fix compiler warning

make[1]: Entering directory '/home/ruben/src/collectd'
  CC       src/utils/oauth/liboauth_la-oauth.lo
src/utils/oauth/oauth.c: In function ‘get_assertion’:
src/utils/oauth/oauth.c:283:19: warning: comparison of integer expressions of different signedness: ‘int’ and ‘size_t’ {aka ‘long unsigned int’} [-Wsign-compare]
  283 |   else if (status >= buffer_size)
      |                   ^~

Merge pull request #3100 from rubenk/virt-fix-compiler-warning

virt plugin: fix compiler warning

Merge pull request #3099 from rubenk/turbostat-fix-compiler-warnings

turbostat plugin: fix two compiler warnings

Merge pull request #3098 from rubenk/ovs-stats-fix-compiler-warning

ovs_stats plugin: fix compiler warning

virt plugin: fix compiler warning

src/virt.c:725:7: warning: no previous prototype for function 'metadata_get_hostname' [-Wmissing-prototypes]
char *metadata_get_hostname(virDomainPtr dom) {
      ^
1 warning generated.

turbostat plugin: fix two compiler warnings

src/turbostat.c:1466:5: warning: no previous prototype for function 'save_affinity' [-Wmissing-prototypes]
int save_affinity(void) {
    ^
src/turbostat.c:1479:6: warning: no previous prototype for function 'restore_affinity' [-Wmissing-prototypes]
void restore_affinity(void) {
     ^

ovs_stats plugin: fix compiler warning

src/ovs_stats.c:863:23: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') [-Wsign-compare]
    for (int i = 0; i < YAJL_GET_ARRAY(ifaces_list)->len; i++) {
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.

Auto-Merge pull request #3097 from octo/ff/build

Automatically merged due to "Automerge" label

configure.ac: Fix include of "utils/mount/mount.h".

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.