1871550 - TPS - Issues: Failure signing and Unable to sync bookmarks and history intermittently

Status: RESOLVED WORKSFORME

(Firefox :: Sync, defect, P3)

Description

[Isabel Rios[:isabel_rios]]

Attachment: Sync Tabs Logs.rtf

Since Dec 20th some of our Sync Integration tests on iOS are failing with different issues. Sharing just one line here and attaching longer logs for each case.

• test_sync_tabs_from_device, fails signing in Nightly
  ERROR: [phase phase1] RunNextTestAction failed - Error: ASSERTION FAILED! Weave status OK; expected "success.status_ok", got "error.login.failed"(resource://tps/logger.sys.mjs:85:13) JS Stack trace: [e-mail]:85:13
  When the test starts with login in Desktop, it works, but not the other way around.

• test_sync_bookmark_from_device, fails checking the item on Nightly
  CROSSWEAVE ERROR: [phase phase1] RunNextTestAction failed - Error: ASSERTION FAILED! places item not found; bookmark (uri: http://www.example.com/, title: Example Domain, location: mobile) not found(resource://tps/logger.sys.mjs:76:11) JS Stack trace: [e-mail]:76:11

• test_sync_history_from_device , fails checking the item on Nightly
  2023-12-22 11:45:13.709 CROSSWEAVE ERROR: [phase phase1] RunNextTestAction failed - Error: ASSERTION FAILED! Uri visits not found in history database: {"uri":"http://www.example.com/","visits":[{"type":1}]}(resource://tps/logger.sys.mjs:76:11) JS Stack trace: [e-mail]:76:11

Comment 1

[Isabel Rios[:isabel_rios]]

Attachment: Sync Bookmark Logs.rtf

Comment 2

[Isabel Rios[:isabel_rios]]

Attachment: Sync History Logs.rtf

Comment 3

[Isabel Rios[:isabel_rios]]

We have also seen issues on the Mobile side, please see issue for your info:
https://github.com/mozilla-mobile/firefox-ios/issues/17901

Comment 4

[Isabel Rios[:isabel_rios]]

Note: For these tests we use latest Nightly version available and Stage server.

Comment 5

[Mark Hammond [:markh] [:mhammond]]

Stage was having problems this week but apparently is now working again - does it work for you?

Comment 6

[Isabel Rios[:isabel_rios]]

Thanks Mark, yes, actually those errors reported that we saw in TPS logs are not happening now.

The only thing that we are seeing is that it takes to long to load the FxA page to enter the email on mobile side.
Less often than when we opened this ticket, we also see the Sync is offline message. I'm attaching screenshots for both things.

We will keep monitoring the tests these days.

Comment 7

[Isabel Rios[:isabel_rios]]

Attachment: Simulator Screenshot - iPhone 15 - 2023-12-26 at 12.11.33.png

Comment 8

[Isabel Rios[:isabel_rios]]

Attachment: Simulator Screenshot - iPhone 15 - 2023-12-26 at 03.01.42.png

Comment 9

[Isabel Rios[:isabel_rios]]

I will update the github FxiOS issue with this info.
If we don't see the TPS errors for a few more days we could close this issue and keep only the one in github.

Comment 10

[Mark Hammond [:markh] [:mhammond]]

we also see the Sync is offline message.

Huh - I don't know how iOS determines that - I don't think any of the sync or fxa components has such a concept, so I suspect that UI is going to be broken in some cases. It may actually be confusing as it's not clear what it's asking the user to do - how are they supposed to put it back online or retry? IOW, I think the fact that UI exists at all is probably a bug on iOS and is likely contributing to some confusion here.

Comment 11

[Isabel Rios[:isabel_rios]]

Thanks for the info Mark, I will ask the iOS devs about that message. Actually, when you see that message, it dissapears after a few seconds, and sync starts, if not, you force sync and then it dissapears.

Comment 12

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:teshaq, could you have a look please?

For more information, please visit BugBot documentation.

Comment 13

[Isabel Rios[:isabel_rios]]

If you agree, we can close this issue. The tests are all passing now so the errors reported here has been resolved.