Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Add metric to bridge routed messages #497

Closed
jrconlin opened this issue Jun 29, 2016 · 0 comments · Fixed by #499
Closed

Add metric to bridge routed messages #497

jrconlin opened this issue Jun 29, 2016 · 0 comments · Fixed by #499
Assignees
@jrconlin
Labels
bug

Comments

@jrconlin
Copy link
Member

For bridged protocols like gcm, need to add metrics indicating successful or error on delivery.

We already log significant events like key changes, delivery failures and deregistration events.
@jrconlin jrconlin self-assigned this Jun 29, 2016
jrconlin added a commit that referenced this issue Jun 29, 2016
@jrconlin
feat: add metrics to bridge protocols
126203b 
This adds metric calls for attempted and successful bridge sends.
format:
        updates.client.bridge.[apns|gcm].[attempted|succeeded]
Errors are already logged.

closes #497
@jrconlin jrconlin mentioned this issue Jun 29, 2016
Merged
bbangert pushed a commit that referenced this issue Jul 13, 2016
@jrconlin @bbangert
feat: add metrics to bridge protocols
e45b82d 
This adds metric calls for attempted and successful bridge sends.
format:
        updates.client.bridge.[apns|gcm].[attempted|succeeded]
Errors are already logged.

closes #497
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jrconlin jrconlin

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add metrics to bridge protocols mozilla-services/autopush
2 participants
@jrconlin @bbangert