tokkee.org

Merge pull request #1740 from rubenk/correct-am_cflags

Append -Werror to the right flags

chrony plugin: constify 2 function arguments

Fixes a couple of dozen of these warnings:
chrony.c:889:20: warning: passing 'const char [14]' to parameter of type
'char *' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
  chrony_push_data("clock_stratum", "chrony", (__extension__ ({ unsigned
short int __v, __x = (unsigned short int)
(chrony_resp.body.tracking.f_stratum); if (__builtin_constant_p (__x))
__v = ((unsigned short int) ((((__x) >> 8) & 0xff) | (((__x) & 0xff) <<
8))); else __asm__ ("rorw $8, %w0" : "=r" (__v) : "0" (__x) : "cc");
__v; })));
                   ^~~~~~~~~~~~~~~

chrony plugin: fix clang warning

chrony.c:745:11: warning: declaration shadows a local variable
[-Wshadow]
      int fh = open("/dev/random", 00);
          ^

collectd.spec: add missing %endif

configure.ac: fix autoreconf warnings

configure.ac:804: warning: AC_LANG_CONFTEST: no AC_LANG_SOURCE call
detected in body
../../lib/autoconf/lang.m4:193: AC_LANG_CONFTEST is expanded from...
configure.ac:804: the top level

Append -Werror to the right flags

testwrapper.sh: Use /bin/sh instead of /bin/bash.

We don't use any bashisms.

Merge pull request #1739 from tokkee/sh/memcheck

unit tests: Run all tests through Valgrind's memcheck, if available.

Actually add the testwrapper.sh script.

… it's now used for running all tests.

configure: Drop unneeded AC_SUBST() calls for AC_PATH_PROG() results.

AC_PATH_PROG calls AC_SUBST for us.

unit tests: Run all tests through Valgrind's memcheck, if available.

collectd.spec: add new gRPC plugin

Disable it for now as the version of protobuf-compiler
in RHEL and Fedora is too old.

ceph_test: Free the yajl handle.

utils_mount_test: Free dynamically allocated strings.

testing: Evaluate expressions only once in EXPECT_EQ_STR().

utils_vl_lookup: Free regexen when destroying class lists.

Else, we'll leak any memory used by the regex.

Merge pull request #1738 from tokkee/sh/silent

src/Makefile: Support silent build rules for pod2man.

src/Makefile: Support silent build rules for pod2man.

Merge pull request #1699 from tokkee/sh/grpc

gRPC server plugin

gRPC doc: Fixed typos and mention C++11 requirement.

src/Makefile: Don't unnecessarily set plugin specific CXXFLAGS.

Also, drop unneeded -lphread / -ldl from grpc's libraries.

src/Makefile: Don't set subdir-objects.

That's now done globally via configure.

collectd.spec: fix rpmlint warning

collectd.src: W: summary-not-capitalized C statistics collection and
monitoring daemon

collectd.spec: make scriptlets work on Fedora

collectd.spec: only require python26 on RHEL < 6

Otherwise the else clause in the condition matches on Fedora too.

This makes the spec build on Fedora.

collectd.spec: disable plugins on older distro

Instead of keeping a list per RHEL version of which plugins to enable,
invert the logic and keep a list of plugins to disable when the distro
is too old. This way we only have to specify a plugin once, and this
also opens up the possibility to build on Fedora.

Merge pull request #1737 from rubenk/pthread-cleanup

Pthread cleanup

collectd.spec: sort list

collectd.spec: EPEL7 has a recent ganglia-devel

collectd.spec: sort plugin lists

collectd.spec: turn TODO into changelog entry

Add new Chrony plugin to rpm spec file

Remove HAVE_PTHREAD_H include guard

We can't build without pthread.h so guarding it is not necessary
anymore.

configure.ac: move AC_SUBST down to after define

Sort types.db

Merge pull request #1548 from bbczeuz/zseng_dev_chrony

[plugin] Chrony NTP service check

src/Makefile.am: merge EXTRA_DIST variables

src/Makefile.am: remove commented out line

src/Makefile.am: remove useless assigment

zfs_arc: simplify build logic

Use BUILD_SOLARIS instead of assuming that any OS not FreeBSD or Linux is Solaris.

src/Makefile.am: remove useless assignment

xencpu: add missing cppflags and ldflags

write_redis: append to CPPFLAGS, not CFLAGS

openvpn plugin: remove useless assignment

oracle plugin: use CPPFLAGS instead of CFLAGS

openldap plugin: use CPPFLAGS instead of CFLAGS

openldap plugin: use $PLUGIN_LDFLAGS

Just like the other plugins.

src/Makefile.am: remove useless assignment

mysql plugin: simplify build logic

The MySQL plugin can't be build without libmysql support.
./configure already takes care of that.

lvm: add missing cppflags and ldflags to Makefile

src/Makefile.am: remove useless assignment

ipvs: simplify logic a bit

src/Makefile.am: remove useless assignment

simplify Makefile.am for nginx and write_http

They can't be build without libcurl support,
./configure will take care of that.

Remove the now unused BUILD_WITH_LIBCURL conditional

Simplify Makefile.am for curl plugins

They can't be build without libcurl support,
./configure will take care of that.

aquaero: add flags to the right variable

src/Makefile.am: merge 2 lines

src/Makefile.am: simplify Apache plugin

It can't be build without libcurl support and
configure will take care of that.

configure.ac: break a few long lines up

Always build with pthread support

Until now you could run ./configure --with-libpthread=no.
This didn't actually compile though.

The daemon has assumed for a while now that pthread support
is available. Bail out in ./configure if this is not the case.

configure.ac: remove unused HAVE_LIBPTHREAD define

unixsock plugin: stop linking with libpthread

The collectd daemon already links with it.

statsd plugin: stop linking with libpthread

The collectd daemon already links with it.

snmp plugin: stop linking with libpthread

The collectd daemon already links with it.

nut plugin: stop linking with libpthread

The collectd daemon already links with it.

notify_email plugin: stop linking with libpthread

The collectd daemon already links with it.

network plugin: stop linking with libpthread

The collectd daemon already links with it.

java plugin: stop linking with libpthread

The collectd daemon already links with it.

exec plugin: stop linking with libpthread

The collectd daemon already links with it.

email plugin: stop linking with libpthread

The collectd daemon already links with it.

dns plugin: stop linking with libpthread

The collectd daemon already links with it.

all plugins: remove pthread.h include

plugin.h already includes pthread.h

Don't hardcode libpthread

collectd_tg: let automake compute dependencies

Automake already does the right thing based on the library added with _LDADD.
This also restores a missing dependency on libheap.la.

collectd_tg: simplify logic a bit

We always link collectdctl with libcollectdclient/libcollectdclient.la.
Make this more obvious.

collectdctl: let automake compute dependencies

Automake already does the right thing based on the library added with _LDADD

collectdctl: simplify logic a bit

We always link collectdctl with libcollectdclient/libcollectdclient.la.
Make this more obvious.

src/Makefile.am: let automake compute dependencies

Automake already does the right thing based on the library
added with _LDADD

src/Makefile.am: simplify logic a bit

We always link collectd_nagios with libcollectdclient/libcollectdclient.la.
Make this more obvious.

Removed braces on single line 'if'. Always using inet_ntop.

Macro fixes, Implemented comments from code review (style, defines)

src/Makefile.am: remove CPPFLAGS assignment

AM_CPPFLAGS are added by default, unless you override _CPPFLAGS

Merge branch 'collectd-5.5'

Conflicts:
src/perl.c

Merge branch 'collectd-5.4' into collectd-5.5

src/Makefile.am: remove unused variable

configure: Add some more information about protoc/gRPC in the summary.

src/Makefile: Move proto-generated sources to nodist_*_SOURCES.

collectd.conf(5): Briefly document grpc's SSL options.

collectd.conf: Update grpc sample configuration.

grpc plugin: Add options to enable SSL protected connections.

grpc plugin: Use std::vector to store listener configs.

grpc plugin: Replace the ListValues RPC with QueryValues.

This is a more powerful function to query values from collectd's cache.

grpc plugin: Split out identifier into it's own message type.

grpc plugin: Use plugin_thread_create instead of pthread_create.

grpc plugin: Always populate *instance fields in a value-list.

That is, ensure a (null-terminated!) empty string if the field is empty.

grpc plugin: Update to the new gRPC API.

grpc plugin: Make RPC call implementation more modular.

Use a template class for the RPC call object implementation to handle all
common functionality. The actual RPC implementation has been moved into
overloaded functions, one for each RPC. This approach only works as long as
the request and response type pairs are unique per call. That's common
practice, though, and in case there's an exception, it would have to fall back
to the previous approach (one class per call).

collectd.conf: Example grpc configuration.

grpc plugin: Implement the ListValues() RPC.

README: Document grpc dependencies.