[collectd-changes] collectd, the system statistics collection daemon: Changes to 'master'

Florian Forster octo at verplant.org
Thu Mar 19 14:21:47 CET 2009


 configure.in         |   10 +++
 src/Makefile.am      |   13 +++
 src/collectd.conf.in |    1 +
 src/types.db         |    1 +
 src/uptime.c         |  218 ++++++++++++++++++++++++++++++++++++++++++++++++++
 5 files changed, 243 insertions(+), 0 deletions(-)

New commits:
commit ec7ae67adc5ef53b67f2b0f91f12f4bc58a799b6
Author: Florian Forster <octo at huhu.verplant.org>
Date:   Thu Mar 19 14:21:16 2009 +0100

    uptime plugin: Shuffle some preprocessor stuff around.
    
    Fix some comments and possibly a Solaris bug, too.

commit d0da1c4e0cb3afe14eda9d24499f7c8bd1692286
Author: Florian Forster <octo at huhu.verplant.org>
Date:   Thu Mar 19 13:51:50 2009 +0100

    build system: Add the new `uptime' plugin.

commit 0c3fcab4abed820084393e64dc30a88512941de1
Author: Marco Chiappero <marco at absence.it>
Date:   Thu Mar 19 13:30:32 2009 +0100

    uptime plugin: Add a plugin to measure the time a system has been running.
    
    Florian Forster ha scritto:
    >Hi Marco,
    
    Hi Florian,
    
    > Checking the KSTAT_TYPE_NAMED twice (three times if you count the
    > assertion) is of course not necessary. I suspect a copy'n'paste error ;)
    
    Me too. Please remove the second one at least, kstat_read do not change
    ks_type.
    
    >A quick `grep' revealed that `get_kstat' is only used in two places, the
    >memory and the swap plugin - both trying to figure out the pagesize. If
    >you need something else than `KSTAT_TYPE_NAMED', I'd suggest to move the
    >checks to a new function `get_kstat_pagesize' and use that function in
    >the two plugins.
    
    I went for the kstat.h only solution.
    Ok, I'm attaching the code, please check the preprocessor directives for
    the includes section, I don't much about defines created by the build
    system and its names.
    I'm using it right now on a linux machine, but I haven't tested yet on
    Solaris and BSDs. Today I tried to compile it on OpenSolaris, but after
    fighting with the packaging system and many other things I gave up and
    decided I won't try anymore. So, since I'd like to say the plugin is
    done after testing it, Solaris testers are wanted :P
    As soon as I can I will edit the wiki page, but first I want to have a
    closer look to the iptable plugin and see how many modifications are
    needed for IPv6 and then have an opinion about the single vs. double
    plugin approach.
    See you soon :)
    
    Regars,
    Marco




More information about the collectd-changes mailing list