Hi Thorsten, On Fri, Apr 30, 2010 at 08:08:04AM -0700, Thorsten von Eicken wrote: > I patched rrdcached.c to accept the same RRD creation options as the > rrdtool plugin accepts. It's not pretty in that it duplicates code, but > it works. Patch for rrdcached.c and collectd.conf.pod below.
Just a quick note, without having looked at the patch or anything ;-) It probably makes sense to provide config handling functions in the `utils_rrdcreate' module for parsing those options. This could then be used in the `rrdtool' and `rrdcached' plugins. Would you be willing to refactor the code in order to implement that? Cheers, Sebastian -- Sebastian "tokkee" Harl +++ GnuPG-ID: 0x8501C7FC +++ http://tokkee.org/ Those who would give up Essential Liberty to purchase a little Temporary Safety, deserve neither Liberty nor Safety. -- Benjamin Franklin
signature.asc
Description: Digital signature
_______________________________________________ collectd mailing list [email protected] http://mailman.verplant.org/listinfo/collectd
