940422 - Isolate telemetry data for each browser front end

Status: RESOLVED WORKSFORME

(Firefox for Metro Graveyard :: General, defect)

Description

[Jim Mathies [:jimm]]

We don't want our performance or other telemetry data intermingling with desktops and vice versa. This bug is about making sure each browser reports an independent data set.

Comment 1

[Jim Mathies [:jimm]]

Looks like this will be ok despite our plans to share a profile. We should get some feedback from telemetry owners though to be sure.

http://mxr.mozilla.org/mozilla-central/source/toolkit/components/telemetry/TelemetryPing.js#650
http://mxr.mozilla.org/mozilla-central/source/browser/metro/metroapp.ini.in

In bug 784389 admin separated out "MetroFirefox" using the name/appid. We should be ok since we're not going to change these values in our app ini file.

Comment 2

[Allison Naaktgeboren :ally]

#telemetry says we need to check with mreid.

Comment 3

[Allison Naaktgeboren :ally]

mreid,
We'd like to confirm that the telemetry data from metrofx is not getting mixed in with the desktop data. We're pretty sure it's fine, but we want to be really sure. I talked to :jonasfj today and he thinks it's probably ok as well, but wants to check with you.

:jonasfj also raises the thought of splitting the metrofx data into a new folder; since we may see significant load when it goes out on GA. (He notes that everything except desktop & fennec end up in one folder).

What are your thoughts?

Comment 4

[Mark Reid [:mreid]]

Currently the Telemetry data layout separates "appName" into: [Firefox, Fennec, Thunderbird, Other].

MetroFirefox is indeed coming through separately from standard desktop Firefox, with an appName of "MetroFirefox", which gets bucketed into "Other".

I agree that splitting MetroFx data into a separate folder is a good idea. Generally, anything we'll want to look at on it's own should be split out from "Other" (as long as it's relatively low-cardinality and not something that will explode the number of folders in the tree).

We can do this as part of bug 933430.

Comment 5

[Allison Naaktgeboren :ally]

Thanks Mark! We'll close this bug and look forward to bug 933430 landing.