stop using obsolete l2met format #58

Merged
merged 1 commit into from Sep 19, 2013

Projects

None yet

3 participants

@mikehale
Collaborator

I have not tested any of this code, but something like this needs to be merged and deployed ASAP so that we can move off the obsolete heroku l2met endpoint.

@ferd
Collaborator
ferd commented Sep 19, 2013

@archaelus thinking of any splunk usage we have that might depend on this format, or any other tool in fact? The fix itself seems safe to merge other than the potential risk of breaking tools that depended on the obsolete l2met format and would be merge-ready otherwise, I think.

@archaelus
Collaborator

Nope - this doesn't bother me in the slightest unless it crashes :) +1

@ferd ferd merged commit 89ad578 into master Sep 19, 2013
@ferd
Collaborator
ferd commented Sep 19, 2013

Tested it locally and all tests keep passing, builds fine.

@mikehale
Collaborator

Splunk search showing obsolete logplex metrics in our l2met.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment