Closed Bug 1552199 Opened 1 year ago Closed 1 year ago

List of created records does not include records from dump on first sync

Categories

(Firefox :: Remote Settings Client, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 69
Tracking Status
firefox69 --- fixed

People

(Reporter: leplatrem, Assigned: leplatrem)

Details

Attachments

(1 file)

If an event listener only treats additions (ie. ignores current), then the records that come from the dump will never be handled.

On the first sync, the list of created records should contain both the ones from the dump and the ones obtain from the sync.

Assignee: nobody → mathieu
Pushed by mleplatre@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6519e3500428
Include records from dump in Remote Settings sync event created data r=glasserc

Backed out changeset 6519e3500428 (Bug 1552199) for test_remote_settings.js failures

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&tochange=4a73759aafa6f3168453b2765f9b13e5fdc640d3&fromchange=6519e35004283dc67e83ce49090480c724fac5bf&selectedJob=247364277

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

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=247364277&repo=autoland&lineNumber=1392

[task 2019-05-20T15:28:57.354Z] 15:28:57 INFO - SUITE-START | Running 1 tests
[task 2019-05-20T15:28:57.978Z] 15:28:57 INFO - TEST-START | services/settings/test/unit/test_remote_settings.js
[task 2019-05-20T15:29:32.169Z] 15:29:32 WARNING - TEST-UNEXPECTED-FAIL | services/settings/test/unit/test_remote_settings.js | xpcshell return code: 0
[task 2019-05-20T15:29:32.170Z] 15:29:32 INFO - TEST-INFO took 34191ms

Flags: needinfo?(mathieu)
Pushed by mleplatre@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a57a160207c4
Include records from dump in Remote Settings sync event created data r=glasserc
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 69
Flags: needinfo?(mathieu)
You need to log in before you can comment on or make changes to this bug.