Add telemetry probes for DataStorage table sizes

RESOLVED FIXED in mozilla37

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mgoodwin, Assigned: keeler)

Tracking

Trunk
mozilla37
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Reporter

Description

5 years ago
DataStorage tables are used for persisting pkp and sts data in nsSiteSecurityService. There's a table size limit after which entries will be evicted.

Telemetry on how close our user population is to this limit would be useful.
Posted patch patchSplinter Review
Assignee: nobody → dkeeler
Status: NEW → ASSIGNED
Attachment #8545436 - Flags: review?(mgoodwin)
Reporter

Comment 2

5 years ago
Comment on attachment 8545436 [details] [diff] [review]
patch

Review of attachment 8545436 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me.
Attachment #8545436 - Flags: review?(mgoodwin) → review+
Had to push a bustage fix. This moves the functionality to a more sane place, anyway. I think I was trying to be too clever with that original patch.
https://hg.mozilla.org/integration/mozilla-inbound/rev/86f3f3bee063
https://hg.mozilla.org/mozilla-central/rev/95d32869e77f
https://hg.mozilla.org/mozilla-central/rev/86f3f3bee063
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in before you can comment on or make changes to this bug.