Merge pull request #4269 from octo/5.12/format_stackdriver

format_stackdriver: When JSON generation fails, reset the generator b…

format_stackdriver: Add proper error handling to `sd_output_reset()`.

format_stackdriver: When JSON generation fails, reset the generator because it is in an unknown state.

write_riemann plugin: Terminate `riemann_event_set` arguments with `RIEMANN_EVENT_FIELD_NONE`.

`riemann_event_set` is a variadic function, that means it accepts a variable
number of arguments. That means it needs some way to determine – at runtime
– how many arguments there are. It appears to be doing so by using
`RIEMANN_EVENT_FIELD_NONE` to indicate the last element in the argument
list. Unfortunately I was unable to find the library's documentation and
code and could not verify this.

That means that the argument list passed to `riemann_event_set` was not
always terminated, causing it to read past where it was supposed to and
adding random crap into the message it crafted.

Issue: #4050

Fix compile of the prometheus plugin

Hugepages plugin skips reading write-only file

Since 'demote' is a write-only file do not attempt to
to read it. This also prevents the plugin from generating
incessant logs about the failure to open it.

Fixes: #3993
https://docs.kernel.org/admin-guide/mm/hugetlbpage.html

virt: Add hugetlb_ metrics

ChangeLog: virt plugin: Add hugetlb_ metrics

Resolves #3932
Co-authored-by: Matthias Runge <mrunge@redhat.com>
(cherry picked from commit 8c781279e9c4789936323670b729f9a979c976db)

[ci][cirrus] Replace trusty with bionic/focal in debian_default_toolchain (#3986)

Ubuntu 14.04 (Trusty) is out of standard support [1].
``make check`` fails for test_capabilities, as noted in [2].
[3] indicates that the cause is glibc, but that updates are not expected
to the version in trusty.

This PR replaces trusty with Ubuntu 18.04 (Bionic) and 20.04 (Focal).

[1] https://wiki.ubuntu.com/Releases
[2] https://github.com/collectd/collectd/pull/3936
[3] https://github.com/collectd/collectd/pull/3927#issuecomment-953350598

(cherry picked from commit b5d8ad13f4e300a7bfe342d27ad804003fcc9173)

Co-authored-by: Emma Foley <efoley@redhat.com>

cirrus: drop CentOS6 (#3985)

It had been reached EOL at November 30th, 2020.
There is no security updates available anymore.

ref. https://wiki.centos.org/About/Product

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
(cherry picked from commit d836b9b20c5ac94ce67f826751d9d717ffabb49c)

Co-authored-by: Kentaro Hayashi <hayashi@clear-code.com>

Fix memory leak in configfile.c

src/netlink.c: remove REG_NOERROR

REG_NOERROR is used since version 5.12.0 and
https://github.com/collectd/collectd/commit/f7fd32e5209e188db7985a42054252fc4cbfe7c1

However, REG_NOERROR is not defined by musl, even GNU regex do not
mention REG_NOERROR, so just remove it to avoid the following build
failure:

src/netlink.c: In function 'check_ignorelist':
src/netlink.c:243:51: error: 'REG_NOERROR' undeclared (first use in this function); did you mean 'REG_NOTBOL'?
       if (regexec(i->rdevice, dev, 0, NULL, 0) != REG_NOERROR)
                                                   ^~~~~~~~~~~
                                                   REG_NOTBOL

Fixes:
 - http://autobuild.buildroot.org/results/cd4d75888b5259b028132dd224be34f69bcbb702

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Merge pull request #3567 from octo/cleanfiles_types_grpc

Makefile.am: Add types.grpc.pb.{cc,h} to CLEANFILES.

Makefile.am: Add types.grpc.pb.{cc,h} to CLEANFILES.

Merge pull request #3566 from octo/test_plugin_intel_rdt

Makefile.am: Link test_plugin_intel_rdt with libpqos.

Makefile.am: Link test_plugin_intel_rdt with libpqos.

Merge pull request #3565 from kwiatrox/release_5_12_kw

Update changelog and version number for release 5.12.

Update changelog and version number for release 5.12.

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

Merge pull request #3564 from octo/snmp_agent

snmp_agent plugin: Fix strncat() truncation warning.

snmp_agent plugin: Fix strncat() truncation warning.

Fixes:

```
src/snmp_agent.c:965:9: error: ‘strncat’ output may be truncated copying between 0 and 127 bytes from a string of length 127 [-Werror=stringop-truncation]
         strncat(out, str, DATA_MAX_NAME_LEN - strlen(out) - 1);
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```

Merge pull request #2470 from dothebart/curl_jolokia_2

Add client interface for the jolokia.org JMX client.

Merge pull request #3549 from octo/infiniband-build

Makefile.am: Remove undefined symbol "BUILD_WITH_INFINIBAND_LIBS".

Merge pull request #3552 from octo/merge-511-512

Merge 5.11 into 5.12

Merge branch 'collectd-5.11' into merge-511-512

Merge pull request #3548 from Self-Perfection/mariadb_10.05_compatibility

mysql: Compatibility with MariaDB >= 10.5

Merge pull request #3543 from dago/fix-network-test

Fix network test

network plugin: Simplify initialization of sockent_t.

Makefile.am: Remove undefined symbol "BUILD_WITH_INFINIBAND_LIBS".

infiniband: fix build

Fix memory leak in curl plugin

This commit will add the patch that fixes the memory leak in curl
plugin. Before when AddressFamily field was set in config file in
variable af was pointer to value of this field and it wasn't freed at
the end. Now this bug is fixed.

mysql: Compatibility with MariaDB >= 10.5

In MariaDB 10.5 information_schema.innodb_metrics was cleaned:
https://github.com/MariaDB/server/commit/d09aec7a15ab4be539d2b110742af544fa6b139f

String column 'status' was replaced with boolean column 'enabled'.

Therefore we need to modify request for fetching data from that table.

Fixes #3533

Merge pull request #3547 from bkjg/fix-leaks-curl

Fix memory leak in curl plugin

Fix memory leak in curl plugin

This commit will add the patch that fixes the memory leak in curl
plugin. Before when AddressFamily field was set in config file in
variable af was pointer to value of this field and it wasn't freed at
the end. Now this bug is fixed.

Empty initializer when gcrypt is absent produces compile error with Sun Studio

Merge pull request #3516 from lukeyeager/fix-ib-build

infiniband: fix build

Merge pull request #3536 from yath/dupcase

Remove duplicate switch case.

Remove duplicate switch case.

The empty 'f' case probably used to be wrapped in its own #if
COLLECT_DAEMON, but that apparently somehow got lost and gcc 9.3
complains about the duplicate case. Therefore, just wrap the
config->daemonize assignment into #if/#endif, like for the other cases.

ChangeLog: collectd: Build fix

Merge pull request #2470 from dothebart/curl_jolokia_2

Add client interface for the jolokia.org JMX client.

daemon: Use EXIT_SUCCESS and EXIT_FAILURE when calling exit_usage().

daemon: Accept the same command line arguments, even when compiled with --disable-daemon.

Ignore -f when compiled with --disable-daemon

Merge pull request #3520 from octo/contributing

docs/CONTRIBUTING.md: Rename "master" to "main".

docs/CONTRIBUTING.md: Rename "master" to "main".

Merge pull request #3518 from pllopis/slurm2002

slurm: compatibility with Slurm >= 20.02

slurm: compatibility with Slurm >= 20.02

infiniband: fix build

Merge pull request #3499 from efuss/netbsd-memory

Add memory plugin implementation for NetBSD

Merge pull request #3487 from kwiatrox/netlink_vf_stats

Netlink plugin: add option to read VF stats

Merge pull request #3476 from ChristianTacke/pr/coll3_types

collection3: Show instance for Current/Voltage

Merge pull request #3497 from efuss/netbsd-entropy

Add entropy plugin implementation for NetBSD

Merge pull request #3500 from efuss/netbsd-processes

Add processes plugin implementation for NetBSD

Merge pull request #3498 from efuss/netbsd-irq

Add irq plugin implementation for NetBSD

Merge pull request #3496 from efuss/netbsd-cpu

Add cpu plugin implementation for NetBSD

Merge pull request #3495 from efuss/netbsd-nfs2

Fix nfs plugin implementation for NetBSD

Merge pull request #3504 from efuss/snmp_count

Add a Count option to snmp plugin

netlink plugin: additional chck for older netlink (kernel) versions

Add a safe check if VF statistics are supported by netlink.

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

Merge pull request #3494 from lukeyeager/infiniband

New plugin: infiniband

infiniband: replace hard-coded constant with variable

infiniband: revert the merge of ib_lid and ib_sm_lid

Merge pull request #3514 from bzed/write_prometheus_lmhttpd

w_prometheus: fix libmicrohttpd api change

Run contrib/format.sh src/write_prometheus.c.

Fixing things I did not touch.

w_prometheus: fix libmicrohttpd api change

Closes #3511

infiniband: run clang-format

infiniband: add more comments to define each attribute

infiniband: consolidate ib_sm_lid and ib_lid

ib_sm_lid becomes type="ib_lid", type_instance="sm"

infiniband: constrain the range of GAUGE types more closely

infiniband: relicense to MIT

Merge pull request #3510 from p-zak/nvme_smart

NVMe extension for SMART plugin

Merge pull request #3512 from pvxe/libmicrohttpd_fix

capabilities: conditionalize mhd callback return type

Fixed CI issues and addressed comments from review

Fixed typo in collectd.conf.pod
Replaced strncasecmp with strcasecmp in smart.c

Added explicit cast to unsigned long before pointer cast in ioctl mock function

va_list struct is now properly closed
fixed typos in smart_test.c
Replaced magic number of intel vid with define
Replaced magic string with const char* with better defined name
grammar fixes in smart.c

capabilities.c: run format.sh

capabilities: conditionalize mhd callback return type

libmicrohttpd introduced an API change with respect its callback
return types. Now its signature says that an enum must be returned
instead of an int. This causes collectd to not finish compilation
because of the compiler warnings this arised (in addition to -Werror)

Add conditional compiling so systems with microhttpd >=0.9.71 can
compile collectd with -Werror

Fixes #3511

NVMe SMART - makefile adjustments

NVMe SMART - makefile adjustments

Add NVMe attributies to SMART plugin

Merge pull request #3502 from efuss/netbsd-netstat_udp

Add a (currently NetBSD-only) netstat_udp plugin

Allow non-DS_COUNTER values for Count option

Allow the value to be of an type other than DS_COUNTER when using the Count option.
I'm sure this can be done more elgantly.

Indentation changes

Mechanical indentation changes intentionally exempted from the last commit (to facilitate review).

Add a Count option to snmp plugin (minus indent)

Add a Count option to the snmp plugin allowing to gather the number of table entries (matching given criteria) rather than their values.
This is especially useful in combination with the recently introduced Filter options.

Example: Count number of total/connected stations on a (Lancom) Access Point.

Indentation changes to existing code are deferred to the next commit in order ease review.

Merge pull request #3503 from paweltox/upstream_intel_rdt_fix_PSWD6320

intel_rdt: correct value metrics reported by the plugin

Add pagesize/getpagesize() for NetBSD case

In the NetBSD case (has sysctlbyname(), but prefers sysctl()), declare pagesize, #include <unistd.h> and call getpagesize().

clang-format

Re-order #includes for the sake of appeasing clang-format. Fortunately, it still compiles that way.

clang-format

Re-order #includes for the sake of appeasing clang-format. Fortunately, it still compiles that way.

clang-format

Re-order #includes for the sake of appeasing clang-format. Fortunately, it still compiles that way.

Merge pull request #3486 from feckert/pr/20200624-ubi

ubi: fix data source type

Merge pull request #3501 from efuss/netbsd-swap

Add swap plugin implementation for NetBSD

clang-format

Apply clang-format.

clang-format

Apply clang-format.

clang-format

Apply clang-format. Makes part of the code harder to read.

clang-format

Partly apply clang-format. Makes part of the code harder to read. Do not re-order sys/types.h

clang-format

Partly apply clang-format. Makes part of the code harder to read. Do not re-order sys/types.h.

clang-format

Partly apply clang-format. Do not re-order sys/types.h.

clang-format

Apply clang-format.

clang-format

Apply clang-format. Makes parts of the code harder to read.

Revert re-ordering of pagesize definition

Revert a re-ordering of #ifdef'd block that may have lead to pagesize being defined, but unused.

Implement swap plugin on NetBSD

Extend swap plugin support for NetBSD, and add per-swap-device reporting for HAVE_SWAPCTL_THREE_ARGS.
Written by Håvard Eidnes <he@NetBSD.org>

Implement processes plugin on NetBSD

Implement the processes plugin on NetBSD.
Written by Håvard Eidnes <he@NetBSD.org>

Implement memory plugin for NetBSD

Add a port of the memory plugin for NetBSD using VM_UVMEXP2, and preferring sysctl over sysctlbyname.
Written by Håvard Eidnes <he@NetBSD.org>