Add telemetry to check if the temporary folder is writable
Categories
(Core :: Audio/Video: GMP, task)
Tracking
()
People
(Reporter: aosmond, Assigned: aosmond)
References
Details
Attachments
(2 files, 1 obsolete file)
48 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
RyanVM
:
approval-mozilla-esr128+
|
Details | Review |
48 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-esr115+
|
Details | Review |
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.
Assignee | ||
Comment 1•1 month ago
|
||
Assignee | ||
Comment 2•1 month ago
|
||
Comment 3•1 month ago
|
||
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.
Comment 5•1 month ago
|
||
bugherder |
Assignee | ||
Comment 6•1 month ago
|
||
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
Comment 7•1 month ago
|
||
Comment on attachment 9430019 [details]
Bug 1923741 - Add telemetry to measure how many users have read only system temporary folders.
Approved for 132.0b7.
Updated•1 month ago
|
Assignee | ||
Updated•27 days ago
|
Assignee | ||
Comment 9•27 days ago
|
||
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.
Assignee | ||
Comment 10•27 days ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D225139
Updated•27 days ago
|
Comment 11•26 days ago
|
||
Comment on attachment 9430019 [details]
Bug 1923741 - Add telemetry to measure how many users have read only system temporary folders.
Approved for 128.4esr.
Updated•26 days ago
|
Comment 12•26 days ago
|
||
uplift |
Comment 13•26 days ago
|
||
Comment on attachment 9431146 [details]
Bug 1923741 - Add telemetry to measure how many users have read only system temporary folders.
Approved for 115.17esr.
Updated•26 days ago
|
Comment 14•26 days ago
|
||
uplift |
Description
•