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

Emit a metric for location lookup failing #120

Closed
pjenvey opened this issue May 28, 2021 · 2 comments · Fixed by #130
Closed

Emit a metric for location lookup failing #120

pjenvey opened this issue May 28, 2021 · 2 comments · Fixed by #130
Labels
enhancement New feature or request

Comments

@pjenvey
Copy link
Member

pjenvey commented May 28, 2021 •
edited by data-sync-user
Loading

We forgot to add a metric for the case of location lookup failing #95.

This isn't critical at the moment but I think it's still worth adding at some point.

┆Issue is synchronized with this Jira Task
@pjenvey pjenvey added the enhancement New feature or request label May 28, 2021
@pjenvey pjenvey mentioned this issue May 28, 2021
Closed
@jrconlin
Copy link
Member

jrconlin commented Jun 3, 2021

Taking the liberty to add a bunch of other metrics as well.

jrconlin added a commit that referenced this issue Jun 3, 2021
@jrconlin
feat: add more metrics
55ecc36 
Adds richer set of metrics and metric handling

Closes #120
@jrconlin jrconlin mentioned this issue Jun 3, 2021
Merged
jrconlin added a commit that referenced this issue Jun 7, 2021
@jrconlin
feat: add more metrics (#130)
786fe72 
* feat: add more metrics

* Adds richer set of metrics and metric handling
* Add stupid GC function for tiles cache.

Issue: #71, #68, #35
Closes #120
@data-sync-user
Copy link
Collaborator

➤ JR Conlin commented:

closed as part of feat: add more metrics #130 ( #130 )

@data-sync-user data-sync-user mentioned this issue Jun 24, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add more metrics mozilla-services/contile
3 participants
@pjenvey @jrconlin @data-sync-user