Skip to content
This repository has been archived by the owner on Mar 25, 2022. It is now read-only.

Prometheus metrics from nginx logs #158

Merged
7 commits merged into from
Mar 28, 2016
Merged

Prometheus metrics from nginx logs #158

7 commits merged into from
Mar 28, 2016

Conversation

ghost
Copy link

@ghost ghost commented Mar 27, 2016

We introduce github.com/google/mtail for parsing nginx logs, and exposing request and response metrics to Prometheus.

  • The build and install scripts are more verbose than they'll eventually have to be. The whole container build, start, restart thing hasn't been generalized yet.
  • Oldnet is being touched for exposing mtail's HTTP interface, and wiring it with Prometheus. These two aspects haven't been migrated to Provsn yet.

TODO: have the gateway dashboard use the metrics scraped from mtail (s/handler="gateway"/job="mtail")

Lars Gierth added 6 commits March 27, 2016 18:00
License: MIT
Signed-off-by: Lars Gierth <[email protected]>
License: MIT
Signed-off-by: Lars Gierth <[email protected]>
License: MIT
Signed-off-by: Lars Gierth <[email protected]>
It's hardwired with only the nginx logs for now.
Needs to a bit more flexibility to deal with further future log files.

License: MIT
Signed-off-by: Lars Gierth <[email protected]>
License: MIT
Signed-off-by: Lars Gierth <[email protected]>
License: MIT
Signed-off-by: Lars Gierth <[email protected]>
@ghost
Copy link
Author

ghost commented Mar 27, 2016

Should probably contribute nginx.mtail upstream, as another example mtail program.

License: MIT
Signed-off-by: Lars Gierth <[email protected]>
@ghost
Copy link
Author

ghost commented Mar 28, 2016

Now taking Content-Type response headers into account too :) @diasdavid

@ghost ghost merged commit 243bfe1 into master Mar 28, 2016
@ghost ghost deleted the nginx-mtail branch March 28, 2016 03:14
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant