Closed Bug 1064480 Opened 10 years ago Closed 6 years ago

Telemetry regression in XPIDB_SAVES_LATE

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED INACTIVE

People

(Reporter: Irving, Unassigned)

References

Details

Attachments

(2 files)

The telemetry regression dashboard at https://tbpl.mozilla.org/?tree=Try&rev=f9e54ceec78f&pusher=ireid@mozilla.com indicates a regression in the SIMPLE_MEASURES_XPIDB_SAVES_LATE measure on the 2014-08-20 nightly build. This measure is 1 when the extensions.json data is dirty and needs to be written out at shutdown time; a regression indicates that something is likely changing add-on state that needs to be stored in the DB during shutdown. At its peak (build IDs 08-25 to 08-31) as many as 75% of shutdowns have late writes, the number drops off again from 09-01 to 09-04 though I'm not sure yet whether that's an artifact of telemetry collection. Nightly 35 is averaging 27% of sessions with a late save (out of 142k submissions) so far. http://telemetry.mozilla.org/#filter=nightly%2F34%2FSIMPLE_MEASURES_ADDONMANAGER_XPIDB_SAVES_LATE&aggregates=multiselect-all!Submissions!Mean!5th%20percentile!25th%20percentile!median!75th%20percentile!95th%20percentile&evoOver=Builds&locked=true&sanitize=true&renderhistogram=Graph
Can't find any relevant code changes that might have triggered this regression; I'd like to land some temporary telemetry to see if Nightly will tell us what's up. If this proves useful, we might want to code up a stack summarizer that gives us a more concise identification of who called into the Addon Manager rather than collecting full (rather large) stacks like this patch does; otherwise I plan to back it out before it graduates from Nightly.
Assignee: nobody → irving
Status: NEW → ASSIGNED
Attachment #8487319 - Flags: review?(dtownsend+bugmail)
(In reply to :Irving Reid from comment #1) > bah, copypasta error on the regression dashboard, it's at > http://vitillo.github.io/cerberus/dashboard/#2014-08- > 20SIMPLE_MEASURES_ADDONMANAGER_XPIDB_SAVES_LATE I can't see that page either :(
Attachment #8487319 - Flags: review?(dtownsend+bugmail) → review+
Here's the stacks collected by Telemetry. Unfortunately, by the time the probe rolled out to Nightly, the spike in late saves had gone away. The rate dropped off again sharply around Sept. 8 / 9 nightly build IDs; it's a bit hard to tell from the dashboard whether the drop correlates better with submission date or build ID. There was a Nightly telemetry experiment running from Aug 14 - Sept. 10, see https://telemetry-experiment.cdn.mozilla.net/ and bug 1052714. I suspect that was the cause, and that there's either a code path that disables and/or removes the experiment at shut down time, or a path that leaves the XPI Provider thinking there are pending changes that need to be cleaned up at shut down. This isn't a major issue (a few extra milliseconds of shut down delay) so I don't think it's worth a lot more time.
Assignee: irving → nobody
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → INACTIVE
See Also: → 1805139
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: