1389991 - test_automigration.js is going to permafail when the Gecko version number is bumped 58 on 2017-09-18

Status: VERIFIED FIXED

(Firefox :: Migration, defect)

Description

[Ryan VanderMeulen [:RyanVM]]

[Tracking Requested - why for this release]: Permafailing xpcshell tests on the next merge day

Looks like the FX_STARTUP_MIGRATION_UNDO_BOOKMARKS_ERRORCOUNT probe expires in 58. What would you like to do with it, Gijs?

https://treeherder.mozilla.org/logviewer.html#?job_id=122804154&repo=try

TEST-UNEXPECTED-FAIL | browser/components/migration/tests/unit/test_automigration.js | checkUndoRemoval - [checkUndoRemoval : 301] Should have reported 0 errors to FX_STARTUP_MIGRATION_UNDO_BOOKMARKS_ERRORCOUNT. - 10230750 == 0

Comment 1

[:Gijs (he/him)]

Ryan,  more accurately shouldn't this track 58? I guess bugzilla doesn't let us do that. :-(


Peter, what's the current status of automigration? Is there any chance we'll be using it at all this year? I'm somewhat inclined to scrap the unused telemetry if not, but if we intend to try to make this usable in the near-ish future I can just extend the probes...

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Yeah, no 58 tracking flags at this point.

Comment 3

[Peter Dolanjski [:pdol]]

(In reply to :Gijs from comment #1)
> Peter, what's the current status of automigration? Is there any chance we'll
> be using it at all this year? I'm somewhat inclined to scrap the unused
> telemetry if not, but if we intend to try to make this usable in the
> near-ish future I can just extend the probes...

We likely do want to use automigration, post 57, yes.  We likely need to eliminate the jank issues first. (which was back-burnered for Photon work)

Comment 4

[:Gijs (he/him)]

Attachment: Bug 1389991 - extend browser data migration telemetry probes so they continue to work on 58 and above,

Review commit: https://reviewboard.mozilla.org/r/171086/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/171086/

Comment 5

[:Gijs (he/him)]

Rebecca, I'm effectively asking to extend these histograms in the face of comment 3. They provide telemetry about if/how we migrate data from other browsers either on first startup of Firefox, or later, at the user's initiative. This will help shape focus on that code and how we improve/change it post-57, when we (hopefully) have cycles to make it more performant and can use it to help set users up with the data they already have on their computer in Firefox automatically.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8899746 [details]
Bug 1389991 - extend browser data migration telemetry probes so they continue to work on 58 and above,

Looks good on Try too.

Comment 7

[[e-mail]]

Comment on attachment 8899746 [details]
Bug 1389991 - extend browser data migration telemetry probes so they continue to work on 58 and above,

https://reviewboard.mozilla.org/r/171086/#review178762

This is an extension request for existing measurements.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3a43ef28808d
extend browser data migration telemetry probes so they continue to work on 58 and above, r=rweiss+418169

Comment 9

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/3a43ef28808d