-
Notifications
You must be signed in to change notification settings - Fork 985
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Data delivery issues #18031
Comments
Currently E2E are failing in different PRs because of the following data delivery issues: 1. Contact request is not delivered to the receiver Example is taken from this PR User A sends CR User A_CR sender.log 2. No manual approval required community stucks in Pending after sending request to join Example taken from this PR User A sends request to join community. As a result community status is not changed to Joined automatically but stucked in Pending. User A_community request sender.log |
@vitvly hi! I have closed this issue and posting here the fresh logs and video from nightly build (12/12/23). User A and User B are members of the same community Steps:
Actual result: part of the messages are lost for User B. On the video you can see that from the last stack of messages User B has not received "Q". Also, the video shows messages history where from previous stack of messages only “Q, I, O, P” out of “Q, W, E, R, T, Y, U, I, O, P” has been delivered for User B. User A sender.zip delivery_bugs.mp4 |
When debugging message reliability we often get the number of messages sent and their IDs but we do not know the content of the messages and the type of message sent. This commit adds debug level logs so that it helps in investigations. ref : status-im/status-mobile#18031 Closes [#18206](status-im/status-mobile#18206)
When debugging message reliability we often get the number of messages sent and their IDs but we do not know the content of the messages and the type of message sent. This commit adds debug level logs so that it helps in investigations. ref : status-im/status-mobile#18031 Closes [#18206](status-im/status-mobile#18206)
so @pavloburykh : I was able to reproduce this exact same issue by sending messages in 1 order and didn't receive them in same order on Device B. As you can see 2, 3 which were sent by Device A are missing in Device B. cc @cammellos |
When debugging message reliability we often get the number of messages sent and their IDs but we do not know the content of the messages and the type of message sent. This commit adds debug level logs so that it helps in investigations. ref : status-im/status-mobile#18031 Closes [#18206](status-im/status-mobile#18206)
When debugging message reliability we often get the number of messages sent and their IDs but we do not know the content of the messages and the type of message sent. This commit adds debug level logs so that it helps in investigations. ref : status-im/status-mobile#18031 Closes [#18206](status-im/status-mobile#18206)
When debugging message reliability we often get the number of messages sent and their IDs but we do not know the content of the messages and the type of message sent. This commit adds debug level logs so that it helps in investigations. ref : status-im/status-mobile#18031 Closes [#18206](status-im/status-mobile#18206)
When debugging message reliability we often get the number of messages sent and their IDs but we do not know the content of the messages and the type of message sent. This commit adds debug level logs so that it helps in investigations. ref : status-im/status-mobile#18031 Closes [#18206](status-im/status-mobile#18206)
When debugging message reliability we often get the number of messages sent and their IDs but we do not know the content of the messages and the type of message sent. This commit adds debug level logs so that it helps in investigations. ref : status-im/status-mobile#18031 Closes [#18206](status-im/status-mobile#18206)
When debugging message reliability we often get the number of messages sent and their IDs but we do not know the content of the messages and the type of message sent. This commit adds debug level logs so that it helps in investigations. ref : status-im/status-mobile#18031 Closes [#18206](status-im/status-mobile#18206)
When debugging message reliability we often get the number of messages sent and their IDs but we do not know the content of the messages and the type of message sent. This commit adds debug level logs so that it helps in investigations. ref : status-im/status-mobile#18031 Closes [#18206](status-im/status-mobile#18206)
When debugging message reliability we often get the number of messages sent and their IDs but we do not know the content of the messages and the type of message sent. This commit adds debug level logs so that it helps in investigations. ref : status-im/status-mobile#18031 Closes [#18206](status-im/status-mobile#18206)
UPDATE (Feb 5, 2024): currently we are facing global data delivery issues that have been caught by our e2e in different PRs. Below are couple examples. cc @cammellos @chaitanyaprem @vitvly 1. Community status has not been updated from Pending to Joined for community request senderExample of PR #18602 Preconditions: User A is an admin of non token gated community which does not require admin's manual approval (hereinafter 'Community') Steps:
Actual result: Community status remains in Pending for User B. Expected result: Community status should change to Joined for User B. User_A_comm_admin.log 2. Message sent in community is not visible to other community memberExample of the PR #18438 Preconditions: User A and User B are members of the same community. Steps:
Actual result: User B does not see a message in Community User_A_comm_message_sender.log |
Looking at logs seems to be the issue of subscribe and unsubscribe order status-im/status-go#4659 . Similar overlap is noticed in fleet nodes which must have caused message not to be delivered.
|
The PR that hopes to address the out-of-order problem identified by @chaitanyaprem : status-im/status-go#4665 |
Thanks @vitvly! Would you mind creating a corresponding mobile PR once go PR will be reviewed? So we could run e2e and check on mobile side? |
Adding another today's example of the message delivery issue in case this is something different from what Pavlo reported: My.Movie.0502.mp4(from 0:15 to 2:04 I'm just waiting for delivery, so you can fast-forward that part) As you can see, some messages are delivered with a delay, some are not delivered at all, and for some delivered messages the delivery status does not change for the sender. |
Yes sure, i remember this has to be done, will do. |
No manual approval required community stucks in Pending after sending request to join when light client is disabled hey. Currently E2E is failing in case when the light client disabled. I was able to reproduce this issue manually, but only when both nodes return from being offline as described in the steps below. Steps:
Actual result:The community is in 'pending' status for My.Movie.1.mp4Expected result:The community is in 'joined' status Additional info:Current issue is also reproducible when Logs:User A logs: User B logs: |
I can't figure out which messages to look at from the logs. Maybe @vitvly or @cammellos can help identify them. But typically in case of a device going offline and coming back to online, the store query must have fetched messages which are missed. |
One more example from e2e tests on the Feb 21 nightly build where light client is disabled. Preconditions:There is a group chat with 3 users. Steps:
Logs:user_A_logcat.log.zip |
In this issue we will continue posting data delivery bugs that are found in mobile develop.
The text was updated successfully, but these errors were encountered: