Skip to content
This repository has been archived by the owner. It is now read-only.

some additions for collectd 5.0.0 #2

Merged
merged 2 commits into from Aug 26, 2011
Merged

Conversation

@donjohnson
Copy link

@donjohnson donjohnson commented Jul 9, 2011

Hi,

I'm currently implementing/testing collectd 5.0.0, and wanted to use this script to get the data into graphite (obviously). It was a bit broken and required a fix to the putval_re regex since the timestamps now have millisecond precision (also I made it greedier at the end of the line). Also, I hacked in support for multiple values being sent in the same metric, and then naming those individual values based on a types.db lookup. I've tested this and it works in my environment, but, I haven't written javascript since 1999 and I also did not put too much thought into the structure or variable names...so if there is a cleaner/more concise way to re-write any of this, feel free!

Thanks!

Don Johnson

@jordansissel
Copy link
Contributor

@jordansissel jordansissel commented Aug 26, 2011

Hmm.. I didn't see this pull request until now, sorry!

jordansissel added a commit that referenced this pull request Aug 26, 2011
some additions for collectd 5.0.0
@jordansissel jordansissel merged commit ca35567 into loggly:master Aug 26, 2011
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.