Skip to content
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

[QA] Cyclic Waiting Dependency while adding multiple sync connections #9969

Closed
2 tasks done
dpapac opened this issue Jul 28, 2022 · 7 comments
Closed
2 tasks done

[QA] Cyclic Waiting Dependency while adding multiple sync connections #9969

dpapac opened this issue Jul 28, 2022 · 7 comments
Labels
Milestone

Comments

@dpapac
Copy link

dpapac commented Jul 28, 2022

Pre-submission Checks

  • I checked for similar issues, but could not find any. I also checked the closed issues. I could not contribute additional information to any existing issue.
  • I will take the time to fill in all the required fields. I know that the bug report may be dismissed otherwise due to lack of information.

Describe the QA issue

Issue seen while testing GH Issue: #9791:

- Desktop-Client Version: 2.10.2daily20220629 (build 7886)
-  Client-OS: Virtualbox Windows 10
-  Server Version: ownCloud 10.10.0

By Removing Sync Connection the behavior of the client still remains as described in the issue #9791

Steps to reproduce the issue

  1. Limit Bandwidth to 20 KB/s (or click really fast or simply have a bad connection, Optional: Enable Logging)
  2. Connect to the server via the client
  3. Choose to manually add Sync Folder Connections
  4. First: add the a /root folder
  5. Second: add a sub-folder of the root folder /root/sub-folder
  6. Third: Add the /root folder again and while it is syncing select the 'Remove folder sync connection' of the 'newly' synced folder
    -> The sync bar of the newly added /root folder is just starting to load
  7. Now the blue pending sync icon should appear at every added sync connection

-> Sync Connection is broken now
A Force Sync does not resolve the issue.
A restart of the client fixes the behavior.

Screenshots

allwaiting

cyclic_waiting

Expected behavior

.

Actual behavior

.

@dpapac dpapac added the QA:team label Jul 28, 2022
@TheOneRing
Copy link
Member

That reminds me on how I intended to clean up that queue code

@jnweiger
Copy link
Contributor

Seen with testpilotcloud 2.11.0daily20220815 (build 8327)

image


Seems the second sync connection waits on the first. After removing the first, the second still does not recover:

image


Workaround:

  • quit/restart the client.
  • then the sync recovers and becomes active and finally finishses with nice green checkmark.

@jnweiger
Copy link
Contributor

Confirmed fixed in 2.11.0-rc3

@saw-jan
Copy link
Member

saw-jan commented Mar 15, 2023

Tested with desktop-client ownCloud 4.0.0.10398-daily20230313 f8e194

❌ Dependency while adding multiple sync connections

Upon adding new sync connection, the previously synced connection will also start to sync and wait for the newly added sync connection to complete

simplescreenrecorder-2023-03-15_16.00.26.mp4

CC @HanaGemela @TheOneRing

@TheOneRing
Copy link
Member

The issue described here was for the folders remaining in the waiting state.
Which apparently doesn't happen any more.

@saw-jan
Copy link
Member

saw-jan commented Mar 15, 2023

Tested with ownCloud 2.11.1 (build 8946) c6dbf3

There is no dependency on syncing

simplescreenrecorder-2023-03-15_17.20.05.mp4

@saw-jan
Copy link
Member

saw-jan commented Mar 15, 2023

There is no dependency on syncing

The difference in this behavior is expected.

With client 4.0.0, all connections sync correctly without being stuck in waiting... state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants