tokkee.org

netapp plugin: Refactored volume usage statistics.

Much of this is like disk, wafl and system statistics before. The
“GetVolumeData” has been renamed to “VolumeUsage” and the
“GetDiskUtil” and “GetSnapUtil” options have been changed, too. The
configuration now looks like this:

    <VolumeUsage>
      Capacity "vol0"
      Capacity "vol1"
      IgnoreSelectedCapacity false
      Snapshot "vol1"
      Snapshot "vol3"
      IgnoreSelectedSnapshot false
    </VolumeUsage>

The code now uses to "ignore lists" to check whether capacity and/or
snapshot information should be collected for a volume. This means the
order in which volumes are listed no longer matters and than you can
use such advanced options as selecting volumes via regular expressions.

Fixed NULL pointer bug.

netapp plugin: Inform the user when he screwed up the disk and system blocks, too.

netapp plugin: Print a notice if all WAFL values have been disabled.

This message is printed if the user did supply a <WAFL /> block but
then disabled all supported values. WAFL collection will be disabled
in this case to increase performance.

netapp plugin: Document the graceful return if statistics are not wanted.

Unfortunately this behavior isn't exactly obvious.

netapp plugin: Refactor system statistics.

Same procedure as before: Instead of using the “service handler”,
create a cfg_system_t pointer if the user wants system statistics. Then
call cna_query_system instead of the service handler.

The “GetSystemPerfData” block has been renamed to “System” and the
“Multiplier” option has been replaced by the “Interval” option.

Merge remote branch 'trenkel/st/netapp' into st/netapp

netapp plugin: Refactor handling of the WAFL data.

Basically the same structure as for the Disk data has been used. The
service handler has been removed and replaced by a call to
“cna_query_wafl”.

The “GetWaflPerfData” block has been renamed to “WAFL” to make the
config file easier to read. The “GetBufCache” config option has been
renamed to “GetBufferCache”. Maybe it should be renamed to
“GetBufferHash”, because that's what the NetApp API uses…?

Merge commit 'octo/st/netapp' into st/netapp

netapp plugin: Changed Sven's email address at his request.

netapp plugin: Refactored reading of disk data.

Instead of obscuring control-flow with generic function pointers, use a
clear and easy to read function hierarchy. All disk-related action now
starts with “cna_query_disk (host)” instead of
“service->handler (host, data, service->data)”.

The “GetDiskPerfData”block has been renamed to “Disks”. All those
blocks start with “Get” and most end with “PerfData”, distracting
from the actual relevant part.

The “Multiplier” option has been replaced by the “Interval” option,
which expects a time in seconds rather than a factor which is multiplied
to the host interval.

netapp plugin: Fix a typo.

Copypasta error in volume usage.

netapp plugin: cna_config_disk: Fix typo.

Preparations for collecting snapshot data,
fixed NULL pointer bug,
removed duplicate error message.

netapp plugin: cna_init: Add some comments.

Don't try to graph operations that aren't licensed
on this filer.

netapp plugin: Improved handling host <Host /> blocks.

Use the new "cf_util_get_string" and "cf_util_get_port_number" functions
and free the host configuration if it fails.

The "Login" option has been split up into the "User" and "Password" options.

netapp plugin: Rename set_global_{perf_,}vol_flag.

Hopefully the new names are more descriptive.

src/configfile.[ch]: Implement "cf_util_get_port_number".

tokyotyrant plugin: Remove no longer needed includes.

src/configfile.[ch]: Implement "cf_util_get_string".

src/common.[ch]: Implement "service_name_to_port_number".

It returns the numeric representation of a service name. The implementation
has been taken from the tokyotyrant plugin.

netapp plugin: Use the "cna_config_" prefix for configuration handling functions.

netapp plugin: Add a short description to most functions.

The order of some functions has been changed, too.

netapp plugin: Add folding markers to all remaining functions.

netapp plugin: Changed some type names.

The structure is roughly like this: Structs that only hold flags to tell
the functions what data to submit are prefixed with "cfg_". Structs that
hold old values for counters are prefixed with "data_".

The "disk_t" type now included flags, too, to indicate valid / invalid
values. The "query_submit_disk_data" function has been changed to honor
those flags.

Various "volume_data" stuff has been renamed to "volume_usage" to make
it more distinguishable from "volume_performance".

Various defines are now also prefixed with "CFG_" to show which flags
are used for configuration and which are used do mark counters valid.
The latter use the "HAVE_" prefix.

AUTHORS: Add Sven Trenkel.

README: Add the “netapp” plugin.

netapp plugin: Split up “collect_perf_wafl_data” …

… into “query_wafl_data” and “submit_wafl_data”.

Again, flags are used to keep track of valid (old) counters.

netapp plugin: Split “collect_perf_volume_data” …

… into “query_volume_perf_data” and “submit_volume_perf_data”. The
functions use the “per_volume_perf_data_t” struct to pass the counters
from one value to the other. The flags have been extended to include
HAVE_* flags. This way we can reliably determine whether an “old”
counter is valid or not.

netapp plugin: Use the "config_get_multiplier" to handle the multiplier configuration.

netapp plugin: process_volume_flag: Use the boolean set/unset option here, too.

netapp plugin: set_global_{perf_,}vol_flag: Make set/unset option a boolean.

netapp plugin: Make "config_init" static.

netapp plugin: build_perf_sys_config: Improve error handling.

Check return value of "malloc".

netapp plugin: Simplify handling of boolean config options.

netapp plugin: perf_system_data_t: Remove CPU counters.

netapp plugin: collect_perf_system_data: Use the submit functions.

netapp plugin: collect_perf_disk_data: Only query "percentage-saved" if required.

Also, add support for "sis_saved_percent == 100". Probably not useful,
but better than simply skipping that case.

Merge commit 'remotes/trenkel/st/netapp' into st/netapp

netapp plugin: collect_perf_disk_data: Add folding markers.

netapp plugin: collect_volume_data: Use "submit_double".

netapp plugin: Use UINT64_MAX as default value.

Meaning "invalid".

netapp plugin: Implement "submit_two_counters".

netapp plugin: Removed INFO

netapp plugin: Added “submit” functions and use them in some other functions.

collectd.conf(5): Add markup to the NetApp plugin documentation.

netapp plugin: Convert C++-style comments to C-style comments.

netapp plugin: Adapt to new dispatch interface.

Build system: Build the netapp plugin when libnetapp is available.

Build system: Implement check for the “netapp” library.

netapp plugin: New plugin to collect statistics from NetApp filers.

Moin,

ich wollts ja eigentlich letzte Woche schon geschickt haben, aber hier ists
nun doch noch: Das collectd netapp Plugin. Es ist noch einiges an doppeltem
Code vorhanden, er ist noch nicht schön und der Configurationscode ist noch
nicht in der angemessenen Ausführlichkeit getestet, aber zumindest hier bei
mir funktioniert jetzt alles und ist voll konfiguriertbar.

Kompilieren tut ich das Ganze so:
gcc -g -c -Wall -I include -I /home/ifst/collectd-4.4.2/src netapp.c
gcc -g -o netapp.so -lnetapp -lxml -lpthread -ladt -lssl -lm -shared netapp.o

Mit freundlichen Grüßen

Sven Trenkel

Merge branch 'collectd-4.8'

Merge branch 'collectd-4.7' into collectd-4.8

perl plugin: Declare the “environ” variable.

processes plugin: Case argument of “isdigit” to int.

On some platforms passing a char there causes problems :/

processes plugin: Remove unnecessary call of realloc(3).

Hi Florian (et al)

> you're right, the (re-)allocation of the memory can probably be avoided
> if the function is turned into one with the following prototype:
> -- 8< --
>  static int *ps_read_tasks (int pid,
>     unsigned long *ret_num_proc,
>     unsigned long *ret_num_lwp);
> -- >8 --

Mmm, why not something like: "static int ps_read_task(pid)"?
This returns the number of task for pid passed as argument.
(AFAIK the function only return the number ot threads), why
we need the ret_num_proc and the ret_num_lwp parameters?

My proposal is attached (code is always cleaner than explanations :P)

Regards,
  Andres

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

Merge branch 'collectd-4.8'

Merge branch 'collectd-4.7' into collectd-4.8

Build system: Check for “libiptc/libip6tc.h” and “linux/netfilter/x_tables.h”, too.

Apparently “linux/netfilter/x_tables.h” is not available with older
kernels which leads to build fails there:
-- 8< --
 In file included from libiptc.c:47,
                  from libip4tc.c:136:
 xtables.h:24:38: linux/netfilter/x_tables.h: No such file or directory
-- >8 --

Merge branch 'collectd-4.8'

Merge branch 'collectd-4.7' into collectd-4.8

src/owniptc/Makefile.am: Don't search KERNEL_DIR for headers.

The iptc library is currenly only enabled, if the required headers where
found without “-I${KERNEL_DIR}”. Adding it to the CFLAGS when building
the shipped version of libiptc just breaks things, for example on
“collectd-master-amd64-linux-2.6”:
  Linux hotdamn 2.6.9-42.ELsmp #1 SMP Tue Aug 15 10:35:26 BST 2006 x86_64 x86_64 x86_64 GNU/Linux

Merge branch 'collectd-4.8'

Merge branch 'collectd-4.7' into collectd-4.8

README: Document the “--with-libiptc=shipped” option.

Build system: Further improve libiptc handling.

Only check for “iptc_handle_t” and “ip6tc_handle_t” if using a
system-wide version of libiptc. If we use the shipped version, we *know*
it provides these types.

Merge branch 'collectd-4.8'

Merge branch 'collectd-4.7' into collectd-4.8

Build system: Improve detection of the iptc library.

When checking for the iptc headers and data types, the configure script
added the kernel directory to the CFLAGS. Later, when actually building
the iptables plugin, the CFLAGS were left untouched.

At least on Debian, the “real” kernel headers are not required – the
libc versions in /usr/include/linux are sufficient. The usage of
KERNEL_DIR has therefore been removed from the iptc checks.

In addition, an directory specified by “--with-libiptc=/path” is no
longer added to the global CFLAGS but rather to the iptables specific
CPPFLAGS.

Hopefully this resolved build problems on various platforms.

contrib/collection3: skip host dirs without read access.

This patch prevents get_files_from_directory() from
failing when DataDir contains directories which we
don't have access to.

An example would be "lost+found" directories, on
ext2/ext3 filesystems, which belong to root and have
0700 filemode.

Signed-off-by: Marc Fournier <marc.fournier@camptocamp.com>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

contrib/collection3: “Invokations” → “Invocations”.

D'oh!

contrib/collection3: Add a vertical label to the “java_memory” type.

contrib/collection3: Added the “invokations” type.

contrib/collection3: Use “{plugin_inst}/{type_inst}” for “{instance}”

… if both are present. Otherwise, use just the one that is available.

contrib/collection3: Add support for the “threads” and “total_requests” types.

Merge branch 'collectd-4.8'

Conflicts:
src/exec.c

Merge branch 'collectd-4.7' into collectd-4.8

collectd-exec(5): Document the environment variables.

exec plugin: Clear the signal block mask before calling exec(2).

exec plugin: Set the “COLLECTD_INTERVAL” and “COLLECTD_HOSTNAME” environment variables.

exec plugin: Avoid a warning when freeing meta data.

contrib/collection3: Suppress warnings about non-existent classes.

ChangeLog: Remove an excess newline.

curl plugin: Add the “MeasureResponseTime” option.

It measures response times. Yes, really! ;)

Signed-off-by: Florian Forster <octo@leeloo.lan.home.verplant.org>

src/collectd.conf.in: Fix entries of renamed plugins.

configure.in: Make it possible to force the shipped version of libiptc.

You can force the shipped version of libiptc using:
  --with-libiptc=shipped

Apparently enough systems include broken versions of that library :/

configure.in: Quote $CFLAGS when assigning to another variable.

ChangeLog: Document the “GETTHRESHOLD” command.

src/utils_threshold.c: Update copyright information.

dns, network, rrdtool plugins: Updated copyright information.

swap plugin: Add copyright information for Stefan Völkel.

bindings/java: Compile and install the “GenericJMX” plugin.

collectd-java(5): Document the “GenericJMX” plugin.

scale target: Fix C90 warning (which is upgraded to an error by default).

Should fix this warning:
-- 8< --
 target_scale.c: In function 'ts_invoke_counter':
 target_scale.c:90: warning: this decimal constant is unsigned only in ISO C90
 target_scale.c:91: warning: this decimal constant is unsigned only in ISO C90
 target_scale.c:93: warning: integer constant is too large for 'unsigned long' type
-- >8 --

Merge branch 'collectd-4.7' into collectd-4.8

Conflicts:
ChangeLog
contrib/collection3/etc/collection.conf
src/df.c
version-gen.sh

Bumped version to 4.7.3; Updated ChangeLog.

Merge branch 'collectd-4.6' into collectd-4.7

Conflicts:
ChangeLog
version-gen.sh

Bump version to 4.6.5.