Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple metrics per logline #55

Open
jordi133 opened this issue Apr 18, 2016 · 2 comments
Open

Multiple metrics per logline #55

jordi133 opened this issue Apr 18, 2016 · 2 comments
Labels

Comments

@jordi133
Copy link
Collaborator

Given the server is running
I registered a component with id "multiMetricComponent" and a metric {"multi-regex" : "some action took (\d+) ms and some other action took (\d+) ms", "metric-keys" : ["a-numerical-metric", "another-numerical-metric"]}
I do a POST with {"log" : "some action took 101 ms and some other action took 202 ms"} to /components/parsingConfiguredComponent/logs on the LogReceiver port
the response should have statuscode 202

@jordi133
Copy link
Collaborator Author

Please adjust the integration tests so that this functionality is also covered there

@jordi133 jordi133 changed the title Multiple loglines per metric Multiple metrics per logline Apr 19, 2016
@jordi133 jordi133 added the ready label Apr 20, 2016
@Neojume
Copy link
Contributor

Neojume commented Apr 20, 2016

Maybe merge with the single regex logparser? Instead of a new name (multi-regex) use the same name(regex) and depending on the number of capture groups it will look for the right number of metric keys. Just a thought, lets develop in parallel and discuss after.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants