docs/review_comments.md: Start document with frequent review comments.

src/daemon/utils_cache_mock.c: Add uc_meta_data_{add,get}_signed_int().

Fixes test_format_stackdriver.

write_stackdriver plugin: Remove redundant error message.

A more detailed error is previously logged by wg_call_timeseries_write.

write_stackdriver plugin: Check "http_code" instead of "status".

write_stackdriver plugin: Check pointer for NULL before dereferencing it.

Makefile.am: s/EXTRA_LTLIBRARIES/noinst_LTLIBRARIES/

9a31008108740bf39d86bdfcb86363a71cad7c0c did not make it into the PR
branch. :\

Merge remote-tracking branch 'github/pr/2472'

Auto-Merge pull request #2955 from octo/linux-2.4

Automatically merged due to "Automerge" label

disk plugin: Drop support for Linux 2.4.

Fix Solaris builds

Change introduced in df135922b19890047ebf725e6b82cd1c174c6d52 (#2810) causes build
failure on Solaris:

src/libcollectdclient/server.c: In function ‘server_multicast_join’:
src/libcollectdclient/server.c:103:9: error: unknown field ‘imr_address’ specified in initializer
         .imr_address.s_addr = INADDR_ANY, .imr_multiaddr.s_addr = sa->s_addr,
         ^
src/libcollectdclient/server.c:103:69: error: ‘struct sockaddr_in’ has no member named ‘S_un’
         .imr_address.s_addr = INADDR_ANY, .imr_multiaddr.s_addr = sa->s_addr,
                                                                     ^
src/libcollectdclient/server.c:102:27: error: missing braces around initializer [-Werror=missing-braces]
     struct ip_mreq mreq = {
                           ^
src/libcollectdclient/server.c:102:27: note: (near initialization for ‘mreq’)

Removed this change as not related to WIN32 platform.

Fix Solaris builds

Change introduced in df135922b19890047ebf725e6b82cd1c174c6d52 (#2810) causes build
failure on Solaris:

src/libcollectdclient/server.c: In function ‘server_multicast_join’:
src/libcollectdclient/server.c:103:9: error: unknown field ‘imr_address’ specified in initializer
         .imr_address.s_addr = INADDR_ANY, .imr_multiaddr.s_addr = sa->s_addr,
         ^
src/libcollectdclient/server.c:103:69: error: ‘struct sockaddr_in’ has no member named ‘S_un’
         .imr_address.s_addr = INADDR_ANY, .imr_multiaddr.s_addr = sa->s_addr,
                                                                     ^
src/libcollectdclient/server.c:102:27: error: missing braces around initializer [-Werror=missing-braces]
     struct ip_mreq mreq = {
                           ^
src/libcollectdclient/server.c:102:27: note: (near initialization for ‘mreq’)

Removed this change as not related to WIN32 platform.

Fix 'make distcheck' (for `master-aggregation` hook)

Change introduced in df135922b19890047ebf725e6b82cd1c174c6d52 (#2810)

This change causes error in 'master-aggregation' hook:

ERROR: files left in build directory after distclean:
./liboconfig.la
./libcommon.la
./libmount.la
./libformat_json.la
./libheap.la
./libignorelist.la
./libavltree.la
./libformat_graphite.la
./liblatency.la
./liblookup.la
./libmetadata.la
./libcmds.la

Auto-Merge pull request #2950 from octo/ff/wireless

Automatically merged due to "Automerge" label

wireless plugin: Add bitrate metric.

src/utils_format_stackdriver.c: Fix initial value of cumulative metrics.

The start value of cumulative metrics is stored in the cache. The first
value of a cumulative metric is then skipped, since Stackdriver required
the interval to be non-zero.

While looking into this, I realized that "absolute" metrics are not
cumulative and changed them accordingly.

Merge pull request #2947 from baruchsiach/fix-build-gpsd

gps plugin: fix build with newer gpsd

gps plugin: fix build with newer gpsd

gpsd version 3.18 changed the prototype of gps_read(). Make the
gps_read() call depend on GPSD_API_MAJOR_VERSION to fix that.

This fixes build failures like:

gps.c: In function 'cgps_thread':
gps.c:144:11: error: too few arguments to function 'gps_read'
       if (gps_read(&gpsd_conn) == -1) {
   ^~~~~~~~

Auto-Merge pull request #2945 from octo/ff/travis

Automatically merged due to "Automerge" label

.travis.yml: Determine number of CPUs at runtime.

Auto-Merge pull request #2944 from octo/ff/typo

Automatically merged due to "Automerge" label

src/daemon/common.h: Fix typo.

Merge pull request #2903 from ccin2p3/documentation/libvirt

add some useful info to libvirt manpage

write_stackdriver plugin: Check message for NULL before calling strdup().

write_stackdriver plugin: Set HTTP request timeout.

Merge branch 'collectd-5.8'

Merge branch 'collectd-5.7' into collectd-5.8

Auto-Merge pull request #2939 from octo/pr2937-backport

Automatically merged due to "Automerge" label

exec plugin: dynamically allocate grname buffer.

Backport of a fix in master.

Fixes: #2696

Auto-Merge pull request #2937 from sreedi/fix-2696

Automatically merged due to "Automerge" label

write_stackdriver plugin: Centralize HTTP calls, add API error handling.

This change implements do_post(), a function for doing POST requests.
In case of a user error (a HTTP 4xx status code) the reply is assumed to
be an error and is parsed accordingly.

Clang format fix.

Made some Code Optimization changes per review.

Auto-Merge pull request #2938 from octo/ff/format_json

Automatically merged due to "Automerge" label

src/utils_format_json.c: Remove chatty debug messages.

src/daemon/plugin.h: Fix typo in comment.

fixed build issues

fixed format issues

Refactored getting effective group id from group name part of fork_child

Various files: Remove mode line.

src/utils_format_stackdriver.c: Address code review comments.

* Declare and initialize variables on the same line where possible.
* Remove modeline.

configure.ac: Re-indent whole AC_PLUGIN list.

configure.ac: Quote all macro args, put "if" and "then" on same line.

Makefile.am: Change "noinst_LTLIBRARIES" to "EXTRA_LTLIBRARIES".

Required fix after rebasing on master.

configure: Set default value for plugin_write_stackdriver.

Otherwise the output is going to be something like this when the plugin
cannot be built, which is ugly:

```
    write_sensu . . . . . yes
    write_stackdriver . .
    write_tsdb  . . . . . yes
```

write_stackdriver plugin: Use the "Label" option inside "Resource" blocks.

The previous implementation was too surprising for users, which is not a good
property for a config option.

collectd.conf(5): Improve write_stackdriver documentation.

liboauth: Make oauth_create() private and remove oauth_create_p12().

The former is not used externally, the latter is not used at all.

Makefile.am: Build libgce ony when liboauth is available.

libgce uses oauth_parse_json_token() from liboauth.

Rename write_gcm to write_stackdriver.

write_gcm plugin: Build only when libyajl 2 is avaiable.

collectd.conf: Improve example configs.

src/utils_oauth_test.c: Fix oauth_create_google_json test.

Makefile.am: Fix rebase errors: source files need src/ prefix.

write_gcm plugin: New plugin for Google Cloud Monitoring.

src/utils_format_gcm.[ch]: Implementation of the Stackdriver Monitoring API v3.

Docs: https://cloud.google.com/monitoring/api/ref_v3/rest/

src/utils_gce.[ch]: Add utility for Google Compute Engine.

These functions allow to autodetect some settings when running on
Google Compute Engine (GCE), for example the Project ID, using the
metadata service.

fixed a missing substitution from merge from master

fix issue 2696 to dynamically allocate grname buffer in increments of _SC_GETGR_R_SIZE_MAX/_SC_PAGESIZE

src/utils_oauth.[ch]: Add utility for OAuth authentication.

Merge pull request #2924 from kevin-laatz-intel/dpdk_path_fix

dpdk: fix dpdk runtime config file path

dpdk: fix dpdk runtime config file path

The config path was changed in DPDK by the commit 'adf1d86  eal: move
runtime config file to new location’. As a result, the dpdk plugins won't
be able to link to the DPDK primary application after this change.

This commit adds a compile time check for the DPDK verison being used and
sets the DPDK runtime config file location accordingly.

Suggested-by: Michael Luo <michael.luo@intel.com>
Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>

Merge pull request #2922 from octo/rootfs_prefix

contrib/docker/rootfs_prefix: Add MIT license header.

contrib/docker/rootfs_prefix: Add MIT license header.

On 2018-09-06 Marc Fournier wrote:

Hello Florian !

Yes, sure, sorry to have omitted this! I'm fine with MIT. Glad this is
useful to others :-)

+1 for adding a default "fallback" licence to COPYING. In case this has
unexpected implications, let's just add the MIT licence to root_prefix/* so
Baruch can move forward with his project.

Cheers,
Marc

Merge branch 'collectd-5.8'

Merge pull request #2919 from rubenk/virt-fix-segfault

Fix segfault in libvirt plugin

virt plugin: fix segfault on error path

The libvirt plugin refreshes its list of domains every RefreshInterval
seconds. If a domain crashes after getting the list, getting the stats
for its block devices returns an error. We then try to call
virDomainGetName on an invalid pointer. Fix this.

fix c_avl_iterator_prev (#2917)

utils_avltree: fix 'c_avl_iterator_prev()' and cover it by tests

Merge pull request #2913 from StarryInternet/sensors-humidity

Add support for humidity in sensors plugin

add humidity for lm_sensors version >= 0x431

Auto-Merge pull request #2916 from rpv-tomsk/collectd-collectd-5.8

Automatically merged due to "Automerge" label

nfs: Fix number of fields for NFSv4

The previous buffer of 64 fields was too short for the current number of
72+2 fields in NFS 4.2, causing the message

  nfs plugin: Unexpected number of fields for NFSv4 server statistics: 62.

This change, originally proposed in [1], fixes that issue in a
forward-compatible manner by reusing the already known number of fields
from the static arrays.

[1]: https://github.com/collectd/collectd/issues/2076#issuecomment-350466628

Merge pull request #2915 from jplitza/fix-2076

nfs: Fix number of fields for NFSv4

nfs: Fix number of fields for NFSv4

The previous buffer of 64 fields was too short for the current number of
72+2 fields in NFS 4.2, causing the message

  nfs plugin: Unexpected number of fields for NFSv4 server statistics: 62.

This change, originally proposed in [1], fixes that issue in a
forward-compatible manner by reusing the already known number of fields
from the static arrays.

[1]: https://github.com/collectd/collectd/issues/2076#issuecomment-350466628

add humidity for lm_sensors version >= 0x402

add support for humidity in sensors plugin

Fix Solaris builds

Trying to fix error:

./build.sh: syntax error at line 78: `TOP_SRCDIR=$' unexpected

Merge pull request #2864 from jimis/libxml2_configure_fix

Add --with-libxml2 option to configure.

Merge pull request #2907 from Stackdriver/pkg-fix

Fix warning that pkgdatadir and pkglibdir were previously defined.

Add --with-libxml2 option to configure.

It's now valid to disable building with libxml2, using --without-libxml2. It's
also possible to set a custom prefix --with-libxml2=/custom/path.

Merge pull request #2896 from mlichvar/chrony

chrony: Ignore late responses

chrony: Ignore late responses

If a response from chronyd was not received before the timeout (e.g. on a
busy system), it would wait in the receive queue and be processed as a
response of the next request. It would fail the sequence check and cause
the plugin to return with an error. The latest response would wait in
the receive queue and this would repeat forever.

Flush the receive queue before sending a new batch of requests to
prevent this situation from happening.

Fix warning that pkgdatadir and pkglibdir were previously defined.

Merge pull request #2904 from rpv-tomsk/redis-connect-unixsock

redis plugin: Added option for connecting via UNIX socket

redis plugin: Added option for connecting via UNIX socket

redis plugin: Fix coredump when stopping Collectd with inaccessible Redis server

add some useful info to libvirt manpage

Change-Id: I321d320885c780f88ffe9f4d9538dad4c6c57e74

Minor codestyle fix

Merge pull request #2898 from romanek-adam/bugfix/memory-leak-in-plugin-dispatch-multivalue

daemon: make plugin_dispatch_multivalue() obey write queue limits

daemon: make plugin_dispatch_multivalue() obey write queue limits (#2897)

dbi plugin: Removed excessive error reporting

Latest libdbi code sets BADIDX error when no more rows is
in 'dbi_result_next_row()' (similar to 'dbi_result_prev_row()'
and to match library documentation).

There is confusing errors in Collectd logs due to this change.
Added a 'dbi_result_has_next_row()' check to solve.

Closes: #2894

Merge pull request #2882 from Stackdriver/gnulib

Add GNULIB_DIR to LDFLAGS in configure.ac on Windows

Merge pull request #2885 from Stackdriver/plugins

Enable additional plugins on Windows.

Enable additional plugins on Windows.

Add GNULIB_DIR to LDFLAGS in configure.ac on Windows

Base port to Windows (#2810)

* Update configure.ac / Makefile.am to build for Windows using Cygwin.
* Update build.sh to build for Windows.
* Base port of the daemon.
* Include gnulib as a submodule.

configure.ac: Split libxml2 and libvirt mix

Merge pull request #2878 from nhuff/freebsd_pending_io

Report number of in progess disk IO requests on FreeBSD.

Fix formating

Report number of in progess disk IO requests on FreeBSD.

Removed parentheses around return arguments