virt: Fix memory leak with libvirt MetadataXPath enabled

Issue: #3225

Merge pull request #3217 from rpv-tomsk/issue-3215

daemon: Check if plugin actually loaded before reporting configuration issues

Merge pull request #3202 from mrunge/collectd-5.9

Collectd 5.9

Special case NaN double values for GAUGE metrics.

JSON does not support NaN values and the Stackdriver POST requests are malformed becuase of cutting the request in the middle.
This change special-cases NaN values and prevents them from being written to the backend.

Clang-format a few files to make CI happy

Another round of clang-format

related to backports/cherry-picks.

Run all changed files 68 8.c/1*.h through format.sh web util

(cherry picked from commit 27ca3ca92a757b5c9bb3baa5a29ef352ec70b1c9)

Run all changed files 68 8.c/1 *.h through clang-format agin..

(cherry picked from commit 626d3ab4309cdb069289354c45ca9e2ba9bc9a2f)

Run all changed files 68 8.c/1*.h through clang-format

(cherry picked from commit eada465ff40eb2ecaef17490e5462dab3b8874fe)

Add snprintf wrapper for GCC 8.2/3

(cherry picked from commit abaa1c8a24e8eff5632dd6052b1da5f6535caf19)

daemon: Recover setlocale() call in src/daemon/collectd.c do_init()

865f2eb31e8264edd9337bc63ac59f6c2fa3b5a7 breaks locale checks

Issue: #3181
(cherry picked from commit 38fac5d5579b952f9546623f39cdbcae89a6f090)

disk.c: remove unused lines

limits.h check removed in 865f2eb31e8264edd9337bc63ac59f6c2fa3b5a7
limits.h included in src/daemon/collectd.h unconditionally

Issue: #3181
(cherry picked from commit d99416997de1c4d9e676960719327a52a364f605)

Merge upstream, address conflict in src/utils/cmds/cmds_test.c

(cherry picked from commit a72ad42076337f3b85db3b06565ae86a83226ab3)

Move Makefile rules for pid_test inside conditional for code

(cherry picked from commit 45b8c056d89f4b029e6017035ce6a4a37a36d260)

Add missing definitions for libnetsnmpagent

(cherry picked from commit 3b05d9f7b7b46e0c4fee296ad2d6f16ecc82173e)

Merge pull request #3224 from mrunge/codeowners

Move trusted-contributors down the list

Move trusted-contributors down the list

since the last matching pattern takes the most precedence.

Merge pull request #3214 from rpv-tomsk/issue-3181

Issue 3181

daemon: Check if plugin actually loaded before reporting configuration issues

Issue: #3215

configure.ac: Removed unused checks for functions

daemon: Recover setlocale() call in src/daemon/collectd.c do_init()

865f2eb31e8264edd9337bc63ac59f6c2fa3b5a7 breaks locale checks

Issue: #3181

disk.c: remove unused lines

limits.h check removed in 865f2eb31e8264edd9337bc63ac59f6c2fa3b5a7
limits.h included in src/daemon/collectd.h unconditionally

Issue: #3181

Merge pull request #3153 from zebity/master

Add snprintf wrapper to avoid "Truncation Warnings" with GCC 8.2/3 compiler

Run contrib/format.sh across entire *.h/*.c source tree (2nd of 2)

Run contrib/format.sh across entire *.h/*.c source tree (1st of 2)

Merge pull request #3072 from octo/ff/network_test

Add a unit test for the network plugin.

Merge remote-tracking branch 'upstream/master'

run all updated *.h *.c through clang-format again

Merge upstream, address conflict in src/utils/cmds/cmds_test.c

Merge upstream, address conflict in src/utils/cmds/cmds_test.c

Auto-Merge pull request #3206 from dago/pid_test

Automatically merged due to "Automerge" label

UBI plugin: enable plugin only for Linux system

UBI plugin: add documentation

Update the README, the collectd.conf.in and collectd.conf.pod

Fix bug that leads to CPPFLAGS gets overridden with CFLAGS when libxxms is enabled

Merge branch 'master' into master

UBI plugin: take account of remarks

* compile the plugin by default
* reword the configure.ac description
* do not touch ChangeLog

Move Makefile rules for pid_test inside conditional for code

Auto-Merge pull request #3205 from rpv-tomsk/3112-powerdns

Automatically merged due to "Automerge" label

powerdns: Added 'unreachables' reporting

Issue: #3112

powerdns: Fix formatting, broken by clang-format

UBI plugin: coding style update

UBI plugin: update the ChangeLog

UBI plugin: collect some stats about the UBIFS

UBIFS is a filesystem for unmanaged flash memory devices.
This plugin collects some stats about the UBIFS:
 * bad physical underlying blocks (bad_peb_count)
 * max erase counter value (max_ec)

Auto-Merge pull request #3203 from dago/netsnmpagent

Automatically merged due to "Automerge" label

Add missing definitions for libnetsnmpagent

dns plugin: Do not use headers from glibc

glibc has removed __NAMESER and __BIND symbols from their codebase.

There is no benefits of use defines from glibc, so replaced them by numbers.

Closes: #3145
(cherry picked from commit faa20ef796abc91656c5c583970db9a9f4daf911)

Fix return code introduced by #3182

Change-Id: I0972f74f3fff05cf29fa9b0be383f0b0df1e6d03
Fixes: #3200
(cherry picked from commit 835a58c09ff7afd44627915c9274b16c5df0fb50)

Use pid_test only when compiling the plugin that uses it

(cherry picked from commit a7401cc81d05bf4f230aaf35e4109a23b60fc546)

clang-format

(cherry picked from commit 4af2afc84a90fe37e4988fca08ea79a6343c6b94)

better user feedback

Change-Id: Iae7c3208024372485fd0901898cbe2e178610082
(cherry picked from commit d75591c9762f93d8656846fd2796fdb775b1c69f)

Fix return value or loglevel for several plugins

Some plugins' RC was != 0 and started to fail due to
stricter verification introduced by 3b9c7b2.
This commit fixes those return values.
For some plugins, fix verbosity of error message: non-zero rc
should be error, not warning.

Change-Id: I9a3f1f80e266858b6744fd9d9d99b352b8d94306

Change-Id: Ibf6ebc6cdc93c6e105d488e4a131dcb6e8eea19b

Change-Id: I35bac15fa0a89b068575739ac1cff0115c9d3a40

s

Change-Id: I992002c56763fbdea5347e5b6e176cc86f5a08ce
(cherry picked from commit a07f44c9cd5828a84545dcfe22544bfed2ea8812)

Use GCC-specific flags only when compiling with GCC

(cherry picked from commit 9317d0734808f56a5d181ff2864332705fcb87e0)

Make sure files in this changeset are formatted properly

(cherry picked from commit 9315ccb08ebd29f0b79426853c2e7adf448c26e9)

Add standard include early or _FILE_OFFSET_BITS will have definition mismatches on Solaris

(cherry picked from commit 54d4f889eb72249824ef4ee6b155b0e0fb858be2)

be more verbose when plugin config cb is failing

3b9c7b2 introduced stricter RC check.
This commit provides user feedback when config callback is failing.

Change-Id: Ia9c13048e95559b5be84477fc1602ff418a1df37
(cherry picked from commit 42a30efe33c4d16276589e0dd1f9e135b38e054d)

regex match: Fix unexpected match with empty meta data

If a user set

  MetaData "foo" "bar"

and a metric does not have meta data (vl->meta == NULL), it causes
unexpected match.

Existance of MetaData config should be checked before meta data in
a metric.

See also: #1930

Signed-off-by: Takuro Ashie <ashie@clear-code.com>
(cherry picked from commit d4f53887527777296b0982af60f4c0fecdb2f0d9)

fix comment

Change-Id: I655dd0b3cb82053f7cc50dec149d4f7cc25ee18b
(cherry picked from commit 2d485cf961eb65d4a55c5c64aa40fdbcca8f41f1)

fix deps

Change-Id: I8199655fcd22082a2f3cfb0e5ab7db289df1a5e7
(cherry picked from commit 9e5b363dda7306b628c1bf91c349ed51f11dbae6)

New upstream plugins

* nvidia_gpu
* write_stackdriver
* pcie_errors

The first 2 are disabled, as I don't know what lib they require
(configure fails)

Change-Id: I56456b418edd3defb3c5620a5ad58f527951309c
(cherry picked from commit 09abfaaeb71957c6039387ad3f5d82803c14ddda)

Merge pull request #3201 from ccin2p3/f/fix-syslog-rc

Fix return code introduced by #3182

Fix return code introduced by #3182

Change-Id: I0972f74f3fff05cf29fa9b0be383f0b0df1e6d03
Fixes: #3200

Merge pull request #3197 from dago/pid_test

Use test_utils_proc_pids only when compiling the plugin that uses it

Use pid_test only when compiling the plugin that uses it

Merge pull request #2725 from RafaelMarinheiro/fix_grpc_query_values

utils_cache: fix pointer reference and meta_data ingestion

Merge pull request #3178 from ashie/fix-match-regex-for-empty-meta

regex match: Fix unexpected match with empty meta data

Change ipstats logic to become configurable for ipv4

Merge pull request #3195 from dago/gccflags

Use GCC-specific flags only when compiling with GCC

Merge pull request #3160 from rpv-tomsk/collectd-uptime-memcached

memcached plugin: Report uptime metric

Auto-Merge pull request #3193 from dago/fileoffsetbits

Automatically merged due to "Automerge" label

Merge pull request #3156 from rpv-tomsk/dns

dns plugin: Do not use headers from glibc

Merge pull request #3182 from ccin2p3/f/config-return-codes

Fix return value or loglevel for several plugins

clang-format

better user feedback

Change-Id: Iae7c3208024372485fd0901898cbe2e178610082

Fix return value or loglevel for several plugins

Some plugins' RC was != 0 and started to fail due to
stricter verification introduced by 3b9c7b2.
This commit fixes those return values.
For some plugins, fix verbosity of error message: non-zero rc
should be error, not warning.

Change-Id: I9a3f1f80e266858b6744fd9d9d99b352b8d94306

Change-Id: Ibf6ebc6cdc93c6e105d488e4a131dcb6e8eea19b

Change-Id: I35bac15fa0a89b068575739ac1cff0115c9d3a40

s

Change-Id: I992002c56763fbdea5347e5b6e176cc86f5a08ce

Use GCC-specific flags only when compiling with GCC

Make sure files in this changeset are formatted properly

Add standard include early or _FILE_OFFSET_BITS will have definition mismatches on Solaris

Run contrib/format.sh as required by github check

battery.c - run clang-format.

Added a fix for new battery notation in sysfs

be more verbose when plugin config cb is failing

3b9c7b2 introduced stricter RC check.
This commit provides user feedback when config callback is failing.

Change-Id: Ia9c13048e95559b5be84477fc1602ff418a1df37

Note that the plugin is FreeBSD only in README

Add ipstats to collectd.conf.in

Add a general note about ipstats in collectd.conf.pod

Change from ips_packets to packets using type_instance

create plugin ipstats

Align with new util library locations

clang formatting

Align with new util library locations

Merge pull request #3175 from ccin2p3/f/collectd-redhat-spec-5.9.0

redhat spec: fix build due to new upstream plugins

regex match: Fix unexpected match with empty meta data

If a user set

  MetaData "foo" "bar"

and a metric does not have meta data (vl->meta == NULL), it causes
unexpected match.

Existance of MetaData config should be checked before meta data in
a metric.

See also: #1930

Signed-off-by: Takuro Ashie <ashie@clear-code.com>

Cast cdtime_t to unsigned long long when necessary

fix comment

Change-Id: I655dd0b3cb82053f7cc50dec149d4f7cc25ee18b

Attempt to align with new util library locations

fix deps

Change-Id: I8199655fcd22082a2f3cfb0e5ab7db289df1a5e7

New upstream plugins

* nvidia_gpu
* write_stackdriver
* pcie_errors

The first 2 are disabled, as I don't know what lib they require
(configure fails)

Change-Id: I56456b418edd3defb3c5620a5ad58f527951309c

clang formatting, again

Bump CI for connectivity plugin

ChangeLog: s/Foster/Forster/

Bump version to 5.9.0; Update ChangeLog.