1236551 - Update the profile migrator to FHR/Telemetry changes

Status: RESOLVED FIXED

(Toolkit :: Telemetry, defect, P1)

Description

[Georg Fritzsche [:gfritzsche]]

The FHR profile migrator code is here:
https://dxr.mozilla.org/mozilla-central/rev/388bdc46ba51ee31da8b8abe977e0ca38d117434/browser/components/migration/FirefoxProfileMigrator.js#189

We need to:
* drop healthreport.sqlite & healthreport/ migration
* Add Telemetry to the profile migration:
  * migrate: datareporting/state.json, datareporting/session-state.json, datareporting/aborted-session-ping, saved-telemetry-pings
  * compat migration: healthreport/state.json
  * drop datareporting/archived

Comment 1

[Georg Fritzsche [:gfritzsche]]

Attachment: Update the profile migrator to the Unified Telemetry changes

This drops the old FHR migrations and specifically migrates 1) datareporting/state.json & session-state.json for the client id & session details as well as 2) healthreport/state.json as a fallback to migrate old client ids.

Comment 2

[Georg Fritzsche [:gfritzsche]]

Attachment: Update the profile migrator to the Unified Telemetry changes

This drops the old FHR migrations and specifically migrates 1) datareporting/state.json & session-state.json for the client id & session details as well as 2) healthreport/state.json as a fallback to migrate old client ids.

Comment 3

[Matthew N. [:MattN] (PM me if requests are blocking you)]

Comment on attachment 8708458 [details] [diff] [review]
Update the profile migrator to the Unified Telemetry changes

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

I *think* it should be fairly straightforward to create a unit test for this like the other tests that use registerFakePath: https://mxr.mozilla.org/mozilla-central/search?find=%2Fbrowser%2Fcomponents%2Fmigration%2Ftests%2Funit%2F&string=registerFakePath

::: browser/components/migration/FirefoxProfileMigrator.js
@@ +220,5 @@
> +        if (subdir && subdir.isDirectory()) {
> +          let stateFile = this._getFileObject(subdir, "state.json");
> +          if (stateFile) {
> +            let dest = createSubDir("healthreport");
> +            stateFile.copyTo(dest, "");

I guess we will still look at healthreport/state.json in some cases?

Comment 4

[Georg Fritzsche [:gfritzsche]]

(In reply to Matthew N. [:MattN] from comment #3)
> Comment on attachment 8708458 [details] [diff] [review]
> Update the profile migrator to the Unified Telemetry changes
> 
> Review of attachment 8708458 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I *think* it should be fairly straightforward to create a unit test for this
> like the other tests that use registerFakePath:
> https://mxr.mozilla.org/mozilla-central/
> search?find=%2Fbrowser%2Fcomponents%2Fmigration%2Ftests%2Funit%2F&string=regi
> sterFakePath

Ah, i actually found test_fx_fhr.js now, updating that.

> ::: browser/components/migration/FirefoxProfileMigrator.js
> @@ +220,5 @@
> > +        if (subdir && subdir.isDirectory()) {
> > +          let stateFile = this._getFileObject(subdir, "state.json");
> > +          if (stateFile) {
> > +            let dest = createSubDir("healthreport");
> > +            stateFile.copyTo(dest, "");
> 
> I guess we will still look at healthreport/state.json in some cases?

It should we an edge-case - we import from that old healthreport/state.json if we don't have datareporting/state.json yet.
If you migrate a profile before that import happened we'd lose the old client id, so i'd rather be careful here.
Presumably we can drop this in the future.

Comment 5

[Georg Fritzsche [:gfritzsche]]

Attachment: Update the profile migrator to the Unified Telemetry changes

Comment 6

[Georg Fritzsche [:gfritzsche]]

Attachment: Update tests to cover Telemetry profile migrator

Comment 7

[Georg Fritzsche [:gfritzsche]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6fe0494d9ed2

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/c08fefe2594f
https://hg.mozilla.org/integration/fx-team/rev/7f066e9a778b

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c08fefe2594f
https://hg.mozilla.org/mozilla-central/rev/7f066e9a778b