Closed
Bug 1772150
Opened 2 years ago
Closed 2 years ago
Broken sync status is not reset after successful sync
Categories
(Firefox :: Remote Settings Client, enhancement)
Firefox
Remote Settings Client
Tracking
()
RESOLVED
FIXED
103 Branch
People
(Reporter: leplatrem, Assigned: leplatrem)
Details
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details | Review |
Steps to reproduce
- Sync fails consistently (eg. >= 10 times).
- Sync is detected as broken
- Sync succeeds
Expected behaviour
Sync should not be "broken" anymore.
Observer behaviour
Sync state is still broken.
Assignee | ||
Comment 1•2 years ago
|
||
Updated•2 years ago
|
Assignee: nobody → mathieu
Status: NEW → ASSIGNED
Pushed by mleplatre@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ebf238f3db94 Reset broken sync status after success r=gbeckley
Comment 3•2 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox103:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 103 Branch
Assignee | ||
Comment 4•2 years ago
|
||
Comment on attachment 9279116 [details]
Bug 1772150 - Reset broken sync status after success r?gbeckley
Beta/Release Uplift Approval Request
- User impact if declined: No impact on users. But without this patch, the Telemetry that we collect about broken synchronization is flawed.
- 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): The change only affects the detection of the broken sync state
- String changes made/needed:
- Is Android affected?: Yes
Attachment #9279116 -
Flags: approval-mozilla-beta?
Comment 5•2 years ago
|
||
Comment on attachment 9279116 [details]
Bug 1772150 - Reset broken sync status after success r?gbeckley
Approved for 102 beta 3, thanks.
Attachment #9279116 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 6•2 years ago
|
||
bugherder uplift |
status-firefox102:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•