On Dec 22, 2009, at 10:56 AM, Silviu Paragina wrote: > On 22.12.2009 18:43, Markus Roberts wrote: >> Sorry about the delay in going over this. We've been in a push to >> get >> ticket based changes for 0.25.2 ready. The patch looks basically >> reasonably, to the extent to which I know anything about RRD (I don't >> know much). >> > No problem, I thought it got lost. Good luck with the release, I had > no > idea you were on a tight schedule. > >> My only cogent concern is if this code will work for people who have >> the older version of RRD (i.e. are these newer bindings version >> specific)? >> >> > I'm not sure, but since it's released bundled in RRDtools, probably > not. > I'll look around and check, otherwise I'll have to decouple metric and > the actual binding used (Luke suggested that in an earlier email but I > didn't thought I was up to the task, still not sure I am as I'm a > newbie > in ruby)
I think it makes more sense just to tell people to upgrade to the rrd- included version. The ruby rrd libraries have always been semi- abandoned, but having something as part of the rrd distro will help considerably. This part of Puppet has also always been underused, and I think it'll be used even less now that the Dashboard provides such better reporting. -- It is a very sad thing that nowadays there is so little useless information. -- Oscar Wilde --------------------------------------------------------------------- Luke Kanies -|- http://reductivelabs.com -|- +1(615)594-8199 -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.
