1169349 - [Metrics] Replace the deviceID field in FTU with IMEI for a special build flag

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System, defect, P1)

Description

[Doug Sherk (:drs) (inactive)]

For Spark users, we want to have a special build flag that replaces the UUIDs generated for the FTU Pings with the IMEI.

This is for the dogfooding purposes.

The IMEI should be available via the telephony interfaces.

Comment 1

[Doug Sherk (:drs) (inactive)]

Tamara, now that we've landed bug 1160483, can you take this? It's our last remaining Spark dependency on the Metrics.

Comment 2

[Tamara Hills [:thills]]

Hi Doug,

I can't do it this week.  There is a chance I can look at it next week, depending on my progress on my other project.

Also, Is this one critical?  You'll still have the usage statistics.  This just gets sent out once the phone is turned on the first time.  With the usage statistics, you'll know within 24 hours that it was turned on (since they cannot turn off the statistics with this build).

Thanks,

-tamara

Comment 3

[Marshall Culpepper [:marshall_law]]

I just had a conversation with Jean, and I'll be able to tackle this early next week after my ServiceWorker bug is wrapping up.

Comment 4

[Jean Gong :jgong]

Marshall, Are you on track to implement this by end of the week?

Comment 5

[Marshall Culpepper [:marshall_law]]

Attachment: use TelemetryRequest.getDeviceID()

Updated to work with Tamara's new TelemetryRequest.getDeviceID() function, which pulls the device IMEI for dogfooders.

Comment 6

[Tamara Hills [:thills]]

Comment on attachment 8621136 [details] [review]
use TelemetryRequest.getDeviceID()

Hi Marshall,

Something funny is going on with TelemetryRequest.  It's giving me an undefined on line 110 when I run it.  Odd, because it's listed in the globals.  I'm wondering if this is a bootstrap.js issue because I moved line 110 outside of the promise and I still get an undefined and the error then points back to bootstrap.js line 19.

-tamara

Comment 7

[Marshall Culpepper [:marshall_law]]

Comment on attachment 8621136 [details] [review]
use TelemetryRequest.getDeviceID()

Hey Tamara, I think I found the issue. I moved the lazy loading of telemetry.js to ensurePing() which happens before both initSettings() and ping() are called. I tested this with and without |make dogfood| on my flame, and the behavior looks right.

Comment 8

[Tamara Hills [:thills]]

Comment on attachment 8621136 [details] [review]
use TelemetryRequest.getDeviceID()

Hi Marshall,

It looks really good.  That fixed the issue.

Thanks,

-tamara

Comment 9

[Marshall Culpepper [:marshall_law]]

https://github.com/mozilla-b2g/gaia/commit/a95d9af4f46edf35b5b9c59ea9bf81529d2ecb2f

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

I had to revert this in https://github.com/mozilla-b2g/gaia/commit/a447f4614001b709477c505bb3e55e44f0a41936 for ftu_ping_test.js failures on Treeherder:

https://treeherder.mozilla.org/logviewer.html#?job_id=2153126&repo=b2g-inbound

Comment 11

[Marshall Culpepper [:marshall_law]]

I have a new PR up with the jshint fix, not sure why I didn't see this the first time around. Will merge once it looks good in treeherder.

Comment 12

[Marshall Culpepper [:marshall_law]]

https://github.com/mozilla-b2g/gaia/commit/25799901335c6c4bfcd9afa1c8748ca23a76e0b7