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

Emit Metrics to Logs #61

Closed
6 of 7 tasks
treeder opened this issue Aug 22, 2016 · 7 comments
Closed
6 of 7 tasks

Emit Metrics to Logs #61

treeder opened this issue Aug 22, 2016 · 7 comments

Comments

@treeder
Copy link
Contributor

treeder commented Aug 22, 2016

Using log format defined here: https://github.com/iron-io/logspout-statsd

Eg:

metric=someevent value=1 type=count
metric=somegauge value=50 type=gauge

Emit the following metrics (at least):

  • wait time total (all apps)
  • wait time per function
  • execution time total (all apps)
  • execution time per function
  • function request count
  • function error count
  • cpu/memory per function ?
@treeder treeder self-assigned this Aug 22, 2016
@treeder
Copy link
Contributor Author

treeder commented Aug 22, 2016

Working on #10 first in order to get wait times...

@treeder treeder added this to the Beta milestone Sep 2, 2016
@henriquechehad henriquechehad self-assigned this Sep 2, 2016
@henriquechehad
Copy link
Contributor

@treeder about the counts, save it in database or just in variable memory?

@treeder
Copy link
Contributor Author

treeder commented Sep 4, 2016

Neither, just log it.

On Fri, Sep 2, 2016, 4:53 PM Henrique Chehad notifications@github.com
wrote:

@treeder https://github.com/treeder about the counts, save it in
database or just in variable memory?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#61 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEoMhhd_gVDoRTJ61ZjyL8lrdaAfH2eks5qmLbwgaJpZM4JqRQK
.

@henriquechehad
Copy link
Contributor

PR: #88

@henriquechehad
Copy link
Contributor

New metric in PR: #89
(execution time total (all apps))

@henriquechehad
Copy link
Contributor

Metrics: wait time total and reserved memory: #61

@seiflotfy
Copy link
Contributor

seiflotfy commented Oct 2, 2016

I think we are good this can be closed

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

No branches or pull requests

3 participants