1772124 - Early hint telemetry

Status: RESOLVED FIXED

(Core :: Networking: HTTP, task, P2)

Description

[Marc Leclair [:mleclair]]

In order to get a better idea of what files we often see through early hints, we should have some telemetry to understand what is going on there.

Comment 1

[Marc Leclair [:mleclair]]

Attachment: Bug 1772124 - Added Glean metrics to track different type of resources for Early Hints r=dragana,mbucher

Comment 2

[Marc Leclair [:mleclair]]

Attachment: data_request_1772124

Comment 3

[Travis Long [:travis_]]

Comment on attachment 9280585 [details]
data_request_1772124

Data Review

1. Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?

Yes, through the metrics.yaml file and the Glean Dictionary.

2. Is there a control mechanism that allows the user to turn the data collection on and off?

Yes, through the standard telemetry opt-out in preferences

3. If the request is for permanent data collection, is there someone who will monitor the data over time?

N/A, collection to end in 6 months (@ December 2022)

4. Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 1

5. Is the data collection request for default-on or default-off?

Default-on

6. Does the instrumentation include the addition of any new identifiers (whether anonymous or otherwise; e.g., username, random IDs, etc. See the appendix for more details)?

No

7. Is the data collection covered by the existing Firefox privacy notice?

Yes

8. Does the data collection use a third-party collection tool?

No

Result

data-review+

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5e3002235320
Added Glean metrics to track different type of resources for Early Hints r=dragana,mbucher,necko-reviewers

Comment 5

[Iulian Moraru]

Backed out for causing multiple failures.
The failures are related to EarlyHintPreloader and yaml.

Push with failures

Failure log py3 failures
Failure log for bp-hybrid bustage
Failure log for windows builds bustages
Failure log for other build bustages

Backout link

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0e2cee29c95d
Added Glean metrics to track different type of resources for Early Hints r=dragana,mbucher,necko-reviewers

Comment 7

[Noemi Erli[:noemi_erli]]

Backed out changeset 0e2cee29c95d (Bug 1772124) for causing bustages in EarlyHintPreloader.cpp CLOSED TREE

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&collapsedPushes=842382&selectedTaskRun=KyP3HdjPQda84kXstad2kg.0&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=0e2cee29c95d40e69caca79d270df9b17192e390

Log: https://treeherder.mozilla.org/logviewer?job_id=382938582&repo=autoland&lineNumber=33177

Backout: https://hg.mozilla.org/integration/autoland/rev/fbb1a17e83a23e5750948cc97c8e4f68ca7bcff9

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8cc6f6f552a3
Added Glean metrics to track different type of resources for Early Hints r=dragana,mbucher,necko-reviewers

Comment 9

[Atila Butkovits]

Backed out for causing failures at browser_103_preload.js.

Backout link: https://hg.mozilla.org/integration/autoland/rev/b267a31087d74520ef7d6cc1d8b1aa6f2e99e1b8

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=8cc6f6f552a38fd6c253f8e420ff6584847aeef4&selectedTaskRun=JFp1UclARJavkABBZxmJ9w.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=383563518&repo=autoland&lineNumber=3214

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/aa5248f0df82
Added Glean metrics to track different type of resources for Early Hints r=dragana,mbucher,necko-reviewers

Comment 11

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/aa5248f0df82

Comment 12

[Julian Descottes [:jdescottes]]

Attachment: Bug 1772124 - [devtools] Remove extra console.log in devtools/server/actors/network-monitor/network-observer.js

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3ea5307a81e2
[devtools] Remove extra console.log in devtools/server/actors/network-monitor/network-observer.js r=devtools-reviewers,bomsy

Comment 14

[Iulian Moraru]

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

Comment 15

[BugBot [:suhaib / :marco/ :calixte]]

A patch has been attached on this bug, which was already closed. Filing a separate bug will ensure better tracking. If this was not by mistake and further action is needed, please alert the appropriate party.