Telemetry shows mozilla sites OK, so it is time to start making them into production (starting with low risk).

Comment on attachment 8429399 [details] [diff] [review] pin-update1 Review of attachment 8429399 [details] [diff] [review]: ----------------------------------------------------------------- ::: security/manager/tools/PreloadedHPKPins.json @@ +176,1 @@ > "pins": "mozilla", "test_mode": true, "id": 2 }, According to bsmedberg and taras, we cannot keep individual telemetry for this bucket (not that it would tell us much anyway). Please remove the id field.

Comment on attachment 8429399 [details] [diff] [review] pin-update1 Review of attachment 8429399 [details] [diff] [review]: ----------------------------------------------------------------- ::: security/manager/tools/PreloadedHPKPins.json @@ +176,1 @@ > "pins": "mozilla", "test_mode": true, "id": 2 }, a). Let's do this in a separate bug. b). What's the relationship between addons.mozilla.net and mozilla.net? As I understand it, the former is necessary for the proper functioning of the browser, and so it can have a specific telemetry id. If the latter is just a cdn, depending on what it serves, I don't think we can give it a telemetry id.

Comment on attachment 8429399 [details] [diff] [review] pin-update1 Ugh. Bugzilla.

(In reply to David Keeler (:keeler) [use needinfo?] from comment #4) > Comment on attachment 8429399 [details] [diff] [review] > pin-update1 > > Review of attachment 8429399 [details] [diff] [review]: > ----------------------------------------------------------------- > > ::: security/manager/tools/PreloadedHPKPins.json > @@ +176,1 @@ > > "pins": "mozilla", "test_mode": true, "id": 2 }, > > a). Let's do this in a separate bug. My bad -- Camilo came by and I asked him to do everything in 1 bug because I didn't want any sites to be double-counted (or left out) in the stats.

Comment on attachment 8429413 [details] [diff] [review] pin-update1 (v2) Review of attachment 8429413 [details] [diff] [review]: ----------------------------------------------------------------- ::: security/manager/tools/PreloadedHPKPins.json @@ +176,5 @@ > "pins": "mozilla", "test_mode": true, "id": 2 }, > { "name": "aus4.mozilla.org", "include_subdomains": true, > "pins": "mozilla", "test_mode": true, "id": 3 }, > { "name": "cdn.mozilla.net", "include_subdomains": true, > + "pins": "mozilla", "test_mode": false}, nit: space after false

Keeping r+ from mmc