Add statsd metrics to login.webmaker.org

RESOLVED FIXED

Status

Webmaker
Login
--
enhancement
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: humph, Assigned: humph)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Similar to bug 862008, let's add statsd metrics collection to login.
(Assignee)

Comment 1

5 years ago
Created attachment 744132 [details] [review]
https://github.com/mozilla/login.webmaker.org/pull/36
Attachment #744132 - Flags: review?(ross)
Attachment #744132 - Flags: review?(kieran.sedgwick)
Comment on attachment 744132 [details] [review]
https://github.com/mozilla/login.webmaker.org/pull/36

Seems okay, though I'm not entirely sure what I'm looking at.  I know statsd is a node daemon for stats aggregation, but I haven't had time to look too deeply into it.
Attachment #744132 - Flags: review?(kieran.sedgwick) → review+
Comment on attachment 744132 [details] [review]
https://github.com/mozilla/login.webmaker.org/pull/36

One note on the PR which is fine to leave in for now if needs be.

Happy to go with whatever you think on that and fix/merge as required.
Attachment #744132 - Flags: review?(ross) → review+
(Assignee)

Comment 4

5 years ago
Staged on master: https://github.com/mozilla/login.webmaker.org/commit/02125c08b15c959503921926116e5d2cec6ac21e
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Attachment mime type: text/plain → text/x-github-pull-request
You need to log in before you can comment on or make changes to this bug.