2000033 - Integrate Glean MLTelemetry into the inference pipeline

Status: RESOLVED FIXED

(Core :: Machine Learning: On Device, enhancement)

Description

[Taimur]

• Update Pipeline Options: Modify the pipeline options to include a flowId.
• Implement Glean Calls: Add the necessary Glean method calls at the correct points in the pipeline to record ML-related telemetry.
• Add Integration Tests: Implement new integration tests to provide coverage for the new method calls.

Comment 1

[Taimur]

Attachment: Bug 2000033: Integrate Glean MLTelemetry into the inference pipeline

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/5989e4930b37
https://hg.mozilla.org/integration/autoland/rev/449fbf293296
Integrate Glean MLTelemetry into the inference pipeline r=ai-ondevice-reviewers,gregtatum,rconcepcion

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/e30793607c53
https://hg.mozilla.org/integration/autoland/rev/b234666ffc39
Revert "Bug 2000033, Bug 2000682: Integrate Glean MLTelemetry into the inference pipeline r=ai-ondevice-reviewers,gregtatum,rconcepcion" for causing perftests bustages.

Comment 4

[Serban Stanca [:SerbanS]]

Reverted this because it was causing perftests bustages.

• Revert link
• Push with failures
• Failure Log
• Failure line: TEST-UNEXPECTED-FAIL | toolkit/components/ml/tests/browser/browser_ml_autofill_perf.js | Uncaught exception in test bound test_ml_generic_pipeline - at chrome://mochitests/content/browser/toolkit/components/ml/tests/browser/head.js:303 - TypeError: metrics.filter is not a function

Please be aware that there are a lot more variations of these bustages as it can be seen here.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/fe96a03c97e8
https://hg.mozilla.org/integration/autoland/rev/1d6863af55c3
Integrate Glean MLTelemetry into the inference pipeline r=ai-ondevice-reviewers,gregtatum,rconcepcion

Comment 6

[amarc]

https://hg.mozilla.org/mozilla-central/rev/1d6863af55c3

Comment 7

[Taimur]

(In reply to Serban Stanca [:SerbanS] from comment #4)

Reverted this because it was causing perftests bustages.

• Revert link
• Push with failures
• Failure Log
• Failure line: TEST-UNEXPECTED-FAIL | toolkit/components/ml/tests/browser/browser_ml_autofill_perf.js | Uncaught exception in test bound test_ml_generic_pipeline - at chrome://mochitests/content/browser/toolkit/components/ml/tests/browser/head.js:303 - TypeError: metrics.filter is not a function

Please be aware that there are a lot more variations of these bustages as it can be seen here.

resolved