tokkee.org

Language and typo fixes.

utils_curl_stats: Drop the nonnull attribute from curl_stats_from_config.

It's not overly useful after all but generates false compiler warnings /
errors on Xenial using CLANG, since we also check the parameter in the
function (which is what we want in the end and there's no risk in actually
calling it with a NULL value).

curl stats: Use cf_util_get_boolean().

curl stats: Simplified some error checks and removed unused code.

configure / curl stats: Check for CURLINFO_APPCONNECT_TIME.

It's available since cURL 7.19 but RHEL5 ships an older version.

collectd.conf(5): Briefly explain MeasureResponseTime vs TotalTime.

curl stats: Use CamelCase identifiers instead of using underscores.

This is more consistent with the usual collectd style but still close enough
to the names used in cURL.

collectd.conf(5): Add documentation for cURL statistics.

Add request specific statistics to all CURL-based plugins.

All metrics supported by curl_easy_getinfo can be retrieved and dispatched for
each request done through CURL. All statistics are optional and disabled by
default.

configure.ac: indent and sort some conditionals

configure.ac: remove obsolete macro

We don't need it anymore. See
http://gnu-autoconf.7623.n7.nabble.com/Bug-389558-autoconf-AC-PROG-GCC-TRADITIONAL-throws-error-in-comparision-td1885.html

chrony plugin: use ANSI function definitions

chrony.c:392:1: warning: function declaration isn’t a prototype
[-Wstrict-prototypes]
 chrony_set_timeout()
 ^
chrony.c: In function ‘chrony_set_timeout’:
chrony.c:392:1: warning: old-style function definition
[-Wold-style-definition]
chrony.c: At top level:

Merge pull request #1758 from rubenk/README

README: use https for collectd.org link

Merge pull request #1759 from rubenk/ansi-function-declaration

uc_get_iterator: use ansi function declaration

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

Bug#1059 : Remove any white spaces in ethstats stat names (affects VMXNet3 interface)

uc_get_iterator: use ansi function declaration

In file included from common.c:37:0:
utils_cache.h:76:1: warning: function declaration isn’t a prototype
[-Wstrict-prototypes]
 uc_iter_t *uc_get_iterator ();
 ^

README: use https for collectd.org link

Merge pull request #1750 from tokkee/sh/cli

daemon: Let all command line options override global options from the config.

configfile: Log errors when trying to access unknown global options.

daemon: Let all command line options override global options from the config.

This replaces 09fb81e130ad70bf0547cec3d3c349ce8cb092cf which provided a
specific solution for the `PIDFile' option. The new solution is generic and
more nicely abstracted without the need for global variables across multiple
modules.

Fixes #366

Merge pull request #1747 from rubenk/write_graphite_check_fd

Write graphite check fd

wg_send_buffer: return error if fd < 0

CID 116310

write_graphite: remove dead assignment

status is immediately overwritten in the next line.

Merge pull request #1746 from rubenk/swrite-check-for-open-fd

swrite: return error if fd is negative

swrite: return error if fd is negative

Merge pull request #1743 from rubenk/apcups-coverity

apcups: ignore result of swrite in net_shutdown

apcups: ignore result of swrite in net_shutdown

CID 37945

Add test logs to .gitignore

automake: disable gzipped tarballs

We only use bz2.

Fixes make distcheck on FreeBSD-11-CURRENT

test -n ""  || find "collectd-5.5.1.1086.g41e7c34" -type d ! -perm -755  -exec chmod u+rwx,go+rx {} \; -o  ! -type d ! -perm -444 -links 1 -exec chmod a+r {} \; -o  ! -type d ! -perm -400 -exec chmod a+r {} \; -o  ! -type d ! -perm -444 -exec /bin/sh /home/ruben/src/collectd/libltdl/config/install-sh -c -m a+r {} {} \;  || chmod -R a+r "collectd-5.5.1.1086.g41e7c34"
tardir=collectd-5.5.1.1086.g41e7c34 && tar --format=posix -chf - "$tardir" | BZIP2=${BZIP2--9} bzip2 -c >collectd-5.5.1.1086.g41e7c34.tar.bz2
tardir=collectd-5.5.1.1086.g41e7c34 && tar --format=posix -chf - "$tardir" | GZIP=--best gzip -c >collectd-5.5.1.1086.g41e7c34.tar.gz
if test -d "collectd-5.5.1.1086.g41e7c34"; then  find "collectd-5.5.1.1086.g41e7c34" -type d ! -perm -200 -exec chmod u+w {} ';'  && rm -rf "collectd-5.5.1.1086.g41e7c34"  || { sleep 5 && rm -rf "collectd-5.5.1.1086.g41e7c34"; };  else :; fi
case 'collectd-5.5.1.1086.g41e7c34.tar.gz collectd-5.5.1.1086.g41e7c34.tar.bz2' in  *.tar.gz*)  GZIP=--best gzip -dc collectd-5.5.1.1086.g41e7c34.tar.gz | tar -xf - ;; *.tar.bz2*)  bzip2 -dc collectd-5.5.1.1086.g41e7c34.tar.bz2 | tar -xf - ;; *.tar.lz*)  lzip -dc collectd-5.5.1.1086.g41e7c34.tar.lz | tar -xf - ;; *.tar.xz*)  xz -dc collectd-5.5.1.1086.g41e7c34.tar.xz | tar -xf - ;; *.tar.Z*)  uncompress -c collectd-5.5.1.1086.g41e7c34.tar.Z | tar -xf - ;; *.shar.gz*)  GZIP=--best gzip -dc collectd-5.5.1.1086.g41e7c34.shar.gz | unshar ;; *.zip*)  unzip collectd-5.5.1.1086.g41e7c34.zip ;; esac
tar: Parse error: SCHILY.acl.default
tar: Parse error: SCHILY.acl.default
tar: Error exit delayed from previous errors.
*** Error code 1

Stop.
make: stopped in /usr/home/ruben/src/collectd

src/Makefile: Add valgrind.FreeBSD.suppress to the dist tarball.

Merge pull request #1742 from tokkee/sh/suppress

Valgrind: Suppress a bogus invalid read on FreeBSD.

Merge pull request #1741 from rubenk/fix-leak-in-disk-plugin

Plug a memory leak in disk plugin

Valgrind: Suppress a bogus invalid read on FreeBSD.

Valgrind reports an invalid read in common.c:parse_value. The error is
supposed to happen in strlen() on a just previously strdup()ed string.

Possibly, this is some sort of alignment issue. The read size is reported to
be 4 on a buffer of size 3.

https://bugzilla.redhat.com/show_bug.cgi?id=678518 may also be related.

Plug a memory leak in disk plugin

Introduced by bdbc066 so no need to backport.
CID 116311

Makefile: Add testwrapper.sh to dist tarballs.

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.