fix compilation errors

change counter type from size_t to uint64_t

change memory allocation style

add checker in average function

cosmetic changes in custom constructor

cosmetical change in linear constructor

change exponential constructor

add comments to header file

add comments to constructor functions

add comments about stroing tree as an array

add license

add percentile calculation

change custom sizes to custom boundaries

change heap allocation to stack allocation

fix code duplication

fixed factor checker

fix attributes

add unused attribute

add static keyword to private functions

add distribution_average()

add distribution_update()

add distribution_clone()

add custom constructor

fix possible memory leaks in build_distribution_from_bucket_array

add exponential constructor

add build_distribution_from_bucket_array function

add checks + add infinity bucket

add linear constructor

add distribution.h

Merge pull request #3508 from octo/metric_t

Implement metric_t and metric_family_t.

src/daemon/metric.[ch]: Improve doc comments.

statsd plugin: Rename "metric_type_t" to avoid a name clash.

format_json: Rename "label_t" to "keyval_t" in test.

This is to avoid a name clash with the type defined in "metric.h".

src/testing.h: Support NULL pointers in EXPECT_EQ_STR().

Changes in "format_graphite" fix the following error:

./src/testing.h:77:56: error: the address of ‘want’ will always evaluate as ‘true’ [-Werror=address]
   77 |              #actual, got__ ? got__ : "(null)", expect ? expect : "(null)");   \

src/testing.h: Support const pointers in CHECK_NOT_NULL().

src/daemon/metric.[ch]: Implement metric_family_append().

src/daemon/metric.[ch]: Add metric_family_metric_reset().

This and metric_family_metric_append() replace the metric_list_{add,reset}
functions.

src/daemon/metric.[ch]: Add metric_label_{get,set,reset}().

label_set_{get,add,reset} have been made static, providing the calling
code with a nicer API.

src/daemon/metric.[ch]: Implement metric_parse_identity().

src/daemon/metric.[ch]: Data types and functions representing metrics.

This implementation closely mirrors the Prometheus / OpenMetrics
protocol buffer.

A "metric family" is a set of metrics, all with the same name (but
different labels and/or label values). The identity is split between the
two types: the "metric family" holds the "name" while the "metric" holds
the labels. Likewise, the "metric family" holds the metric type, the
"metric" holds the metric value.

This commit contains all required changes to compile the daemon, but
pretty much everything else still fails to build. The tests have not yet
updated either.

Merge pull request #3509 from octo/avltree

avltree: Return errno values; unify argument handling.

avltree: Return errno values; unify argument handling.

Merge pull request #3507 from octo/remove-absolute

Tree-wide: Remove support for "absolute" metric values.

Tree-wide: Remove support for "absolute" metric values.

Merge pull request #3481 from octo/strbuf

strbuf: Library that implements printing to a dynamically allocated buffer.

strbuf: Update buf->size if realloc succeeds.

strbuf: Implenet "strbuf_printn" and "strbuf_print_escaped".

common: Use the strbuf library in format_values().

common: Add test for format_values().

strbuf: Add STRBUF_CREATE_FIXED, unify naming.

This changes the macro and function name to use "fixed" when a buffer and
its size are provided, and "static" if `sizeof()` is used to determine the
size.

The macros now return and expect a `strbuf_t`, i.e. *not* a pointer. This
makes it easier to see that the buffer is allocated on the stack and may
go out of scope.

The buffer size in the tests has been changed to 9, so that `sizeof(array)`
and `sizeof(pointer)` return different values.

strbuf: Improve documentation of exported functions.

strbuf: Add tests for stack allocation.

Add tests for the STRBUF_CREATE[_STATIC] macros. This surfaced a bug with
the STRBUF_DESTROY macro, which was missing `… while (0)` at the end.

strbuf: Library that implements printing to a dynamically allocated buffer.

Merge pull request #3493 from octo/configfile

src/daemon/configfile.c: Remove special case for the "libvirt" plugin.

src/daemon/configfile.c: Remove special case for the "libvirt" plugin.

Merge pull request #3263 from duckfez/http_output_fix

write_http no longer writes http response to stdout

Merge pull request #3463 from miesi/master

mysql: report slave_io_running and slave_sql_running

run format.sh to comply with coding style

replace long if else with compact code

replace slave_running with generic bool

mysql: report slave status running io and sql

Merge pull request #3458 from octo/man-types-db

types.db(5): Improve wording and document conventions.

types.db(5): Improve wording and document conventions.

Merge pull request #3398 from mbakhoff/sensu-source

write_sensu: add the IncludeSource option

Merge pull request #3443 from octo/sensors-man

collectd.conf(5): Document the format expected by "sensors"' ignore list.

Merge pull request #3452 from maciejsszmigiero/mqtt-reconnect-fix

MQTT: Resubscribe after a reconnect

Fix code style issues in src/mqtt.c

This is an automated code style fix generated by running
"contrib/format.sh src/mqtt.c".
No functional change intended.

Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>

MQTT: Resubscribe after a reconnect

After losing our connection to a MQTT broker our subscription will be gone,
too, so we need to subscribe to our topic once again after reconnecting,
otherwise collectd will not receive any MQTT messages anymore.

Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>

Merge pull request #3450 from hightea/zk-add-mntr-fields

Zookeeper - Add missing 3.5 mntr fields

Zookeeper - Add missing 3.5 mntr fields

Add missing zk_last_proposal_size , zk_min_proposal_size and zk_max_proposal_size mntr fields (available since zk 3.5) to handled fields

Merge pull request #3446 from F4-Group/handle-full-stat-name

Add target in category for varnish 6

Add backend name in vbe stat

Split stat key on dot, extract target for SMA/SMF and use it in category

For example, this adds v0 and Transient for SMA, allowing to monitor
cache size properly

Fixes https://github.com/collectd/collectd/issues/2206

Merge pull request #3445 from F4-Group/varnish6-support

Add varnish 6 support

Restrict workers stats to versions before v6

Update counters for MSE 3.0 - varnish plugin

Add counter for vmod-goto, only for Varnish Plus 6

VSM only apply for Varnish 4 or 5

Add varnish 6 new values

Extracted from https://github.com/collectd/collectd/pull/3360

Merge pull request #3410 from mbakhoff/amqp-failover

amqp: allow multiple hosts for failover

amqp: improve example conf

Merge pull request #3441 from maciejsszmigiero/disk-plugin-udev-rule

Provide an udev rule with ID_SERIAL based, disk plugin usable ID attribute.

Update conf files

Add varnish 6 support

Merge pull request #2730 from viulian/master

adding mosquitto_loop after mosquitto_publish, otherwise qos=1 messag…

collectd.conf(5): Document the format expected by "sensors"' ignore list.

This question recently came up on the mailing list.

Provide an udev rule with ID_SERIAL based, disk plugin usable ID attribute

Disk plugin UdevNameAttr option is very useful for making sure that the
same disk always ends in the same RRD file no matter which particular
device node it ends being assigned by the OS.

However, using the most fitting ID_SERIAL udev attribute for this purpose
results in data about the whole disk and each partition being mixed
together incorrectly, since this attribute has exactly the same value for
the whole disk device node and its particular partition nodes.

Let's introduce an ID_COLLECTD attribute instead which differentiates
between the whole disk and its particular partitions in a new udev rule
file provided in "contrib" directory.
To warn users about the issue let's also add a relevant note to the
UdevNameAttr option description.

Based on Stewart Adam's workaround from
https://github.com/collectd/collectd/issues/823

Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>

Code files don't need execute permission

Some .c files had execute permission bit set needlessly.

Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>

Merge pull request #3439 from ffontaine/master

src/dpdk_telemetry.c: fix build on musl

src/dpdk_telemetry.c: fix build on musl

Include unistd.h instead of sys/unistd.h to fix the following build
failure on musl:

src/dpdk_telemetry.c:43:10: fatal error: sys/unistd.h: No such file or directory
 #include <sys/unistd.h>
          ^~~~~~~~~~~~~~

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

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

Merge pull request #2841 from elfiesmelfie/feat_mdevents

New plugin to read RAID events

Merge pull request #3432 from ryan-mccabe/master

amqp1: Add options to limit send queue length

clang-format

write_http: improved logging of server response

amqp1: Add option to limit internal send queue length

ChangeLog: amqp1 plugin: Add option to limit internal send queue length.

Add a config  option to control the internal plugin send queue
length: SendQueueLimit

If SendQueueLimit is set to an integer greater than 0, the send queue
in the amqp1 plugin will be constrained to that number of entries to
avoid unbounded memory use in the case that there is no amqp1
connection. If a new message is queued when the queue has reached
maximum length, the oldest message in the queue will be discarded.

Signed-off-by: Ryan McCabe <rmccabe@redhat.com>

Merge pull request #3433 from collectd/collectd-5.9

Fix memory leak in write_riemann

Merge branch 'http_output_fix' into http_fix_2

Merge pull request #1 from collectd/master

Pull in upstream master

Merge pull request #3425 from rpv-tomsk/openvpn

openvpn plugin: Fix parsing empty fields

Merge pull request #3427 from kkepka/update-master-with-5.11-changelog

Update master with 5.11 changelog