Closed Bug 1923741 Opened 1 month ago Closed 1 month ago

Add telemetry to check if the temporary folder is writable

Categories

(Core :: Audio/Video: GMP, task)

task

Tracking

()

RESOLVED FIXED
133 Branch
Tracking Status
firefox-esr115 --- fixed
firefox-esr128 --- fixed
firefox132 --- fixed
firefox133 --- fixed

People

(Reporter: aosmond, Assigned: aosmond)

References

Details

Attachments

(2 files, 1 obsolete file)

In bug 1922798, users have reported not being able to update Widevine/OpenH264 plugins because their temporary directory was set to read only. This is problematic because many things in Firefox use the temporary folder and those failures are likely difficult to attribute. We should get some telemetry data to get a sense of how many users actually have this problem.

Attached file Data review request, v1 (obsolete) —
Attachment #9430018 - Flags: data-review?(chutten)

Comment on attachment 9430018 [details]
Data review request, v1

Actually, for projects whose instrumentation is added in patches reviewed in Phabricator, there's a new way for conducting data review that doesn't involve forms. Just follow the instructions from the Herald message and you'll be good to go.

Attachment #9430018 - Flags: data-review?(chutten)
Pushed by aosmond@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6d5d8fd20e74 Add telemetry to measure how many users have read only system temporary folders. r=jrmuizel
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 133 Branch

Comment on attachment 9430019 [details]
Bug 1923741 - Add telemetry to measure how many users have read only system temporary folders.

Beta/Release Uplift Approval Request

  • User impact if declined/Reason for urgency: Need to assess how many users are impacted by a read-only system temporary directory. Sooner this probe makes it to release, the sooner we can make a decision.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Just adds a new telemetry probe on startup to check if the temporary folder is writable.
  • String changes made/needed:
  • Is Android affected?: No
Attachment #9430019 - Flags: approval-mozilla-beta?

Comment on attachment 9430019 [details]
Bug 1923741 - Add telemetry to measure how many users have read only system temporary folders.

Approved for 132.0b7.

Attachment #9430019 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Attachment #9430018 - Attachment is obsolete: true

Comment on attachment 9430019 [details]
Bug 1923741 - Add telemetry to measure how many users have read only system temporary folders.

ESR Uplift Approval Request

  • If this is not a sec:{high,crit} bug, please state case for ESR consideration: First telemetry results from nightly show very few but non-zero users with read only temp folders, but ESR may have a different pattern.
  • User impact if declined: May miss there is a significant population with unwritable temp folders
  • Fix Landed on Version: 133
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Just adds a telemetry probe that we have verified in nightly.
Attachment #9430019 - Flags: approval-mozilla-esr128?
Attachment #9431146 - Flags: approval-mozilla-esr115?

Comment on attachment 9430019 [details]
Bug 1923741 - Add telemetry to measure how many users have read only system temporary folders.

Approved for 128.4esr.

Attachment #9430019 - Flags: approval-mozilla-esr128? → approval-mozilla-esr128+

Comment on attachment 9431146 [details]
Bug 1923741 - Add telemetry to measure how many users have read only system temporary folders.

Approved for 115.17esr.

Attachment #9431146 - Flags: approval-mozilla-esr115? → approval-mozilla-esr115+
See Also: → 1925163
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: