Merge pull request #2969 from rpv-tomsk/issue-2173

write_prometheus: Added 'Host' option.

Merge pull request #2970 from octo/ff/oauth

src/utils_oauth.c: Renew OAuth tokens 30 seconds before they expire.

Added missing include of "collectd.h"

This adressed to solve compilation issue on Solaris platform:

./src/config.h:1526:0: error: "_FILE_OFFSET_BITS" redefined [-Werror]

write_prometheus: Added 'Host' option.

This allows to set specific address to listen on.

Closes: #2173
Closes: #2288

src/utils_oauth.c: Renew OAuth tokens 30 seconds before they expire.

This solves sporious errors like this one:

    write_stackdriver plugin: POST https://monitoring.googleapis.com/v3/projects/{projectId}/timeSeries:
      API error 401: Request had invalid authentication credentials.
      Expected OAuth 2 access token, login cookie or other valid authentication credential.
      See https://developers.google.com/identity/sign-in/web/devconsole-project.

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

ovs_stats: use correct OpenFlow value

The correct value in the OpenFlow 1.4 statistics is 1024_to_1522_packets,
even if collectd is using 1024_to_1518_packets:

$ ovs-ofctl -O OpenFlow14 dump-ports ovsbr0
OFPST_PORT reply (OF1.4) (xid=0x2): 3 ports
  port  guest2: rx pkts=0, bytes=0, drop=0, errs=0, frame=?, over=?, crc=?
           tx pkts=0, bytes=0, drop=0, errs=?, coll=?
           duration=132.213s
           rx rfc2819 1_to_64_packets=0, 65_to_127_packets=0,
           128_to_255_packets=0, 256_to_511_packets=0,
           512_to_1023_packets=0, 1024_to_1522_packets=0,
           1523_to_max_packets=0,

Correct it to the right value.

Signed-off-by: Matteo Croce <mcroce@redhat.com>

cleanup: Use compound literals for SO_REUSEADDR setsockopt

Merge pull request #2803 from rorysexton/dev_branch

cpufreq plugin: Read number of p-state transitions and time spent in each p-state

cpufreq plugin: Address code review comments

cpufreq plugin: Address code review comments

cpufreq plugin: Reviewed p-state statistics code

cpufreq: Review time_in_state metric

cpufreq: Review p-state transitions metric

Clang format

Addressing comments on previous commit.

Updating cpufreq.c from clang-format

Extending cpufreq plugin to read the following additional metrics:
  - number of p-state(cpu frequency) transitions
  - time spent in each p-state

Both metrics are reported by the Linux cpufreq-stats driver:
https://www.kernel.org/doc/Documentation/cpu-freq/cpufreq-stats.txt

ovs_stats: add macro to populate counters list

Use a stringification macro to populate the counter array. Doing this
instead of duplicating the tokens is less error prone.

Signed-off-by: Matteo Croce <mcroce@redhat.com>

docs/review_comments.md: Add paragraph "Define variables on first use".

Merge pull request #2806 from rorysexton/dev_turbostat_plugin

turbostat plugin: New metrics: P-states,Turboboost,Platform TDP,Uncore bus ratio

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

statsd: Set SO_REUSEADDR on listening socket

Closes: #1126
Closes: #1187

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.