make DATA_MAX_NAME_LEN configurable at compile time

pinba plugin: fix warning on Solaris

[libprotobuf WARNING google/protobuf/compiler/parser.cc:547] No syntax
specified for the proto file: pinba.proto. Please use 'syntax =
"proto2";' or 'syntax = "proto3";' to specify a syntax version.
(Defaulted to proto2 syntax.)

grep -q is not portable

Solaris grep doesn't have -q

Use the grep we detected with AC_PROG_EGREP and just redirect stdout.

Fix failing tests after 6fd0d25

Fixes #1833

Merge pull request #1832 from rubenk/check-for-c99-compiler

Require a compiler that understands C99

src/daemon/plugin.h: Enlarge the DATA_MAX_NAME_LEN.

Fixes: #966, #1238

dpdkstat: README instructions for DPDK static lib

Updated README with instructions on how to build and link to the static
DPDK library for dpdk plugins.

Change-Id: I580cccc8af648c270c5a35689dc536a283f1689a
Signed-off-by: Korynkevych, RomanX <romanx.korynkevych@intel.com>
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

Modify DPDK CFLAGS

The failed compile when there are no additional
flags on Ub14.04 is specific to GCC 4.8 and below.

This can be fixed by installing GCC 4.9, or
alternately by adding -mssse3 or newer ISA flag.

The opensource community have requested that we
simply document the GCC version requirement,
rather than adding GCC ISA flags to configure.ac

Change-Id: I8ec7bd3f094b0dd608381068fd6b18a74050d420
Signed-off-by: Kim Jones <kim-marie.jones@intel.com>

Require a compiler that understands C99

Upcoming changes will requires C99 support.

For newer compilers this is the default, but older
compilers needs a flag like -std=gnu99 or -std=c99.

Merge pull request #1830 from rubenk/move-collectd-header

treewide: add blank line below collectd.h

Merge pull request #1831 from rubenk/ai_hints-cleanup

Ai hints cleanup

treewide: Use AF_UNSPEC consistently

.ai_family expects an address family, not a protocol family.
Note that on most platforms these constants are the same, so this
is purely cosmetic.

no need to initialize result ptr of getaddrinfo

treewide: use designated initializers for ai_hints

xencpu plugin: Remove unnecessary semicolons.

grpc plugin: Rename "Reply" messages to "Response".

This appears to be the common nomenclature for gRPC.

grpc plugin: Create a "Dispatch" service and use streaming RPCs.

This patch, unfortunately, does two things:
* It breaks out the DispatchValues() call into a separate service. The
  intention of this is to make it easier to implement only part of the
  API, namely to not implement querying of values, which may not be
  useful for stateless proxies.
* It turns the DispatchValues() call into a (client side) streaming RPC.
  That means that a client can send many ValueLists to the server in one
  streaming request, rather than many self-contained requests.

This code is heavily influenced by test code of the "protobuf-qml"
project, which is MIT licensed.

grpc plugin: Add the "ds_names" field to the ValueList message.

grpc plugin: Unify field names.

Fields holding a ValueList are now called "value_list", fields holding
a Value are now called "value". Repeated fields use the plural form.

treewide: stop checking for AI_PASSIVE

I think it's safe to assume that if you have getaddrinfo, you have
AI_PASSIVE. My copy of Stevens from 1998 already describes it.

treewide: stop checking for AI_ADDRCONFIG

It has been in RFC 3494 since 2003 and all the platforms
we care about have it.

treewide: add blank line below collectd.h

Fix build on Solaris 10

Fix result if no protoc 3 found

Merge pull request #1829 from rubenk/clang-format

Add a config file for clang-format

Merge pull request #1821 from rubenk/memset

treewide: replace memset to 0 with initializers

pf: no need to zero-initialize status

The kernel bcopies properly sized data into this struct

Fix double declaration

ipmi: fix building with older gcc's

Add a config file for clang-format

To use it, use `clang-format --style=file`

Try if this fixes the build on RHEL{5,6}

ceph plugin: no need to zero the whole array

also use '\0' not 0 while we're here

ipmi: use C99 designated initializer

apache: no need to zero-initialize

amqp: use C99 designated initializers

amqp: no need to zero-initialize buffer

all messages written into it are null-terminated

aggregration: no need to zero-initialize

treewide: replace memset to 0 with initializers

Merge pull request #1822 from rubenk/write_riemann-indent

write_riemann plugin: reindent

Merge pull request #1824 from rubenk/protobuf-autoconf

Protobuf autoconf adjustments

Use pkg-config to configure grpc plugin

- Check for libgrpc++ instead of libgrpc
- Add check for libprotobuf
- Add configure options --with-libgrpc++, --with-libprotobuf and --with-libprotobuf-c
- Add precious variable PROTOC for path to the protoc compiler
- Add precious variable PROTOC_C for path to the protoc-c compiler
- Add precious variable GRPC_CPP_PLUGIN for path to the grpc_cpp_plugin binary

Fixes #1817

write-riemann plugin: reindent with clang-format

clang-format -i --style=llvm src/write_riemann.c

Merge branch 'pr/1791'

interface plugin: Remove NULL config key.

cpusleep plugin: formatting changes

remove mode lines from interface plugin

interface plugin: following active/inactive interfaces via ReportInactive

Stop using `which` for finding python interpreter

Which is not available in minimal build environments (see #1825 for an
example) so replace it with AC_PATH_PROG.

inteface plugin: add formatting string

Merge pull request #1825 from ripienaar/missing_which

collectd.spec: el7 fails to find python without which

collectd.spec: el7 fails to find python without which

On very minimal build centos machines the yum-builddep should install
all it needs to succesfully build on centos, but this misses which and
so python cant be found.

grpc plugin: Simplify error handling a bit.

Make control flow more straight forward and handle all cleanup in one place.

proto/collectd.proto: Remove unused import "google/protobuf/timestamp".

This causes problems when generating Go code from the .proto.

Merge pull request #1816 from octo/grpc-free-iter

grpc plugin: Free the cache iterator when returning due to an error.

grpc plugin: Free the cache iterator when returning due to an error.

Fixing configure.ac to reflect changes in collectd

Signed-off-by: Kim Jones <kim-marie.jones@intel.com>

Added ASLR info to README

Applied an old version the last time; reverted and reapplying new
version.

Signed-off-by: Kim Jones <kim-marie.jones@intel.com>

Interface name and number of ports enabled can now be customised

Using the .conf files
They are optional arguments

Change-Id: Icc8dd0933f98c3ba05cc6bc3f338070d38b06a05
Signed-off-by: Kim Jones <kim-marie.jones@intel.com>

dpdkstat: refactored pipe error checking

This commit cleans up and fixes the pipe creation, and improves the
error checking.

Change-Id: I53de4a7ff450aea349971f58fe5cce8904da1e89
Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>

configure: check AVX support

Check AVX support before adding -mavx flag.

Change-Id: I2de2a12704126590ef991de89880ad5bd5c3805e
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

dpdkstat: configuration improvements

Cleanup detection and configuration for dpdkstat to include:
* -mavx flag which is need for compilation on Ubuntu 14.04.
* Simplification of code to detect DPDK library.
* Support for DPDK not being installed in /usr/lib.

Change-Id: I2767797f1928934a8545a602b8af92c9decf24d3
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

dpdkstat: cleanup of code

Cleaned up code to ensure appropriate spacing and moved integer declarations
out of loops as it's not supported out of box on C99 compilers.

Change-Id: Ia420fcb5e0a5ce9110fcdc7cc09aaa01c2b86771
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

dpdkstat: fix missed rework

Update README to refer to libdpdk instead of dpdk. Update the
declaration of dpdk_init to remove whitespace.

Change-Id: Ib2b02929bf7ab164fc97a84da1dbb0c7d77a2080
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

dpdkstat: rebase against latest master

Change-Id: I2528cd3cae66a0d2ded0950fec9a529545b75c81
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

dpdkstat: fix configure.ac and src/Makefile

Fix configuration.ac and src/Makefile based on comments from pull
request review.
  * Removed unnecessary libs.
  * Removed unnecessary compilation flags.
  * Fixed indentation in configure.ac
  * Remove duplicate AC_CHECK_LIB check.
  * Compare FOUND_DPDK with xtrue explicitly.
  * Change standalone instances of dpdk to libdpdk.
  * Add BUILD_WITH_DPDK_LIBS to dpdkstat_la_LIBADD.

Change-Id: I27b0c888f81cac3219fdc97174e31770dbd5e746
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

dpdkstat: fix typo in collectd.conf.pod

Fix typo in collectd.conf.pod.

Change-Id: I0de9582c0a3f2731194b73c85aa83a1e6bdc9aec
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

dpdkstat: fix compilation errors

Fix compilation errors for dpdkstat.

Change-Id: I5f20219344f36a12d4a60f2bad6ce8228e3e992e
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

README: update to include min version of DPDK

Update the readme to include the minimum version of DPDK required for
the dpdkstat plugin.

Change-Id: I51ca20284d90586e24bb9711de0b40d4027ca367
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

types.db: fix alignment for dpdkstats

Change-Id: I00173c9a07a288a4aca5912687627b8f62d64f65
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

v2: dpdkstat: fixed issues from github review

v2: fixed indentation of part

This commit fixes all code issues in dpdkstat.c

Mostly error handling, and some minor nit-picks.

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>

dpdkstat: rebased + fixed alignment in types.db

Rebased against the latest master and cleaned up alignment in types.db.

Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

readme: update to include DPDK.

Include DPDK as part of the README features.

Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

dpdkstat: bugfix missed enabled_port_count

Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>
Reviewed-by: Harry Van Haaren <harry.van.haaren@intel.com>

dpdkstat: convert snprintf uses to ssnprintf

Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

dpdkstat: bugfix enable_port_count

Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>
Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>

dpdkstat: fix configuration without DPDK

Modify the configuration in the situation where DPDK is not selected as
an option.

Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

dpdkstat: enable a plugin for DPDK stats

This patch enables support to retrieve statistics for DPDK
interfaces. An overview of the threading is as follows:
1. collectd init() or read() calls dpdk_helper_spawn() to retrieve
   the required size of xstats to allocate.
   1.1. DPDK counts ports, stats and length, writing them to
        shared-memory (SHM).
   1.2. DPDK helper (secondary) process quits, allowing cleanup of
        the shared memory.
2. collectd resizes shared-memory to size of stats as provided by
   DPDK.
3. collectd respawns the DPDK helper.
   3.1. Helper blocks on a semaphore until told to read the stats
        from DPDK and write them to SHM.
4. collectd dispatches statistics
   4.1. Thread blocks on semaphore until stats are available.
This threading model is required to allow the plugin to detect when
the DPDK primary process/application has been killed/reset, and to avoid
the plugin from stopping another DPDK primary process from starting.
Some extra housekeeping is in place to ensure collectd is never
stalled by using sem_timedwait() with a timeout. If collectd dies, the
helper process will automatically quit after a timeout, as it detects
its ppid has changed.

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

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

empty_counter match: Code cleanup.

empty_counter match: Add support for the DERIVE data source type.

Fixes: #1813

varnish plugin: Add informative warnings to the config handling.

Not all config options are valid for all versions of Varnish. Rather
than removing the if-clause altogether, which results in a "Ignoring
unknown configuration option" warning, print a user-friendly warning
instead.

Fixes: #1790

Merge pull request #1806 from rubenk/network-plugin-size_t

network plugin: use size_t for length and offsets

network plugin: Don't abort() if gcrypt initialization failed.

Instead, report an error and let plugin initialization fail.

write_http: fix warning in new kairos support

Found by LLVM:

comparison of integers of different signs: 'int' and 'const size_t' (aka 'const unsigned long')

write_http: remove superfluous braces

Merge pull request #1809 from beorn-/write_http_kairosdb_format_master

Add KAIROSDB format to write_http plugin

Add KAIROSDB format to write_http plugin

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

Conflicts:
ChangeLog
contrib/redhat/collectd.spec
version-gen.sh

Merge pull request #1804 from rubenk/madwifi-fix-buffer-overflow

madwifi plugin: fix buffer overflows

metadata: add comment about metadata functions use and threads safety

Signed-off-by: Florian Forster <octo@collectd.org>

Bump spec file to 5.4.3

Update spec file to 5.5.2

(cherry picked from commit 34add34b27c6d85ed992efc951fc08994e5f86d6)

Update spec file to 5.5.2

Merge pull request #1807 from matteocontrini/master

Added 'operations_per_second' (redis) type

Bump version to 5.4.3; Update ChangeLog.

network plugin: Fix error message for GCRYCTL_INIT_SECMEM failure.

Merge branch 'collectd-5.5'

ChangeLog: Add note about #1665.

Added 'operations_per_second' (redis) type