Reset Sync option 3, doesn't properly propagate to other clients.

VERIFIED FIXED in 1.3b5

Status

()

VERIFIED FIXED
9 years ago
22 days ago

People

(Reporter: tracy, Assigned: Mardak)

Tracking

({regression})

unspecified
1.3b5
regression
Points:
---
Dependency tree / graph
Bug Flags:
blocking-weave1.3 +
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [weave1.3])

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
Filing as a follow up to bug 563443. as seen on 1.3b4

0) A and B are in sync.  
1) On B, disconnect weave, then remove a bunch of bookmarks from the toolbar.
2) Reset sync on Client A with option 3, Replace all other devices with your
local data. let sync complete.
3) Run sync on client B.

Tested results:
The bookmarks that were removed there are not restored by the sync.

Expected results:
The sync from client A should have restored the deleted bookmarks on client B.

Also,  further syncs of both machines A and B does not restore the deleted
bookmarks.  The clients remain out of sync.
Flags: blocking-weave1.3?
(Assignee)

Comment 1

9 years ago
Created attachment 443699 [details] [diff] [review]
v1
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attachment #443699 - Flags: review?(mconnor)
(Assignee)

Updated

9 years ago
Depends on: 556454, 534218, 526569
Keywords: regression
(Assignee)

Comment 2

9 years ago
bug 526569: moved wipeRemote before client sync, so recipient list may be unknown
bug 534218: wipeRemote happens after remoteSetup, so syncID change to force data upload doesn't work
bug 556454: client engine doesn't have a pref to enable, so they weren't syncing

Updated

9 years ago
Attachment #443699 - Flags: review?(mconnor) → review+

Comment 3

9 years ago
good candidate for crossweave unit tests.
Flags: in-testsuite?
(Assignee)

Comment 4

9 years ago
http://hg.mozilla.org/labs/weave/rev/7a66c793d465
Always sync client data by making sure it's always enabled and only update local cilent with remote commands. Make sure to sync clients to get a list of clients that need to receive commands. Also, make sure to upload data after wiping remote now that remoteSetup happens before wipeRemote.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Flags: blocking-weave1.3? → blocking-weave1.3+
Resolution: --- → FIXED
Target Milestone: --- → 1.3
(Assignee)

Updated

9 years ago
Target Milestone: 1.3 → 1.3b5
(Assignee)

Updated

9 years ago
Blocks: 562122
(Reporter)

Comment 5

9 years ago
verified with 1.3b5
Status: RESOLVED → VERIFIED

Comment 6

8 years ago
Investigating this bug for being potential crossweave automation test case
candidate.
Added crossweave test for this as http://hg.mozilla.org/automation/crossweave/rev/0ffcc5f9307f
Flags: in-testsuite? → in-testsuite+
Component: Firefox Sync: UI → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.