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

versions tests sometimes fail with too little versions listed #5853

Closed
individual-it opened this issue Sep 29, 2021 · 5 comments
Closed

versions tests sometimes fail with too little versions listed #5853

individual-it opened this issue Sep 29, 2021 · 5 comments
Assignees
Labels

Comments

@individual-it
Copy link
Member

webUIFilesActionMenu/versions.feature:36 sometimes fails in the step And the versions list should contain 2 entries because only one version could be found
grafik
e.g. https://drone.owncloud.com/owncloud/web/19297/12/14

@jasson99
Copy link
Contributor

jasson99 commented Oct 4, 2021

This has been more frequent now. This should be fixed soon

@individual-it
Copy link
Member Author

those tests have been deleted in #5973 but I think that was a mistake @SwikritiT please bring the tests back that were deleted in #5973 but are still valid
In the same PR you can try to fix the timing issue similar to #5883
We need to understand if that is an issue in the backend or in our testing code
If its a backend issue the tests can stay skipped but we need a core and/or ocis issue to track the problem

@kulmann
Copy link
Contributor

kulmann commented Nov 29, 2021

Do you have any findings, yet, if this is a backend issue or a CI timing issue? Since it's flaky I'd be fine skipping it with reference to this issue.

@fschade is already taking care of skipping it... would be nice to get an actual fix though.

@individual-it
Copy link
Member Author

@kulmann I thing the function that was waiting between uploads (because oc 10 only stores one version per sec) had an issue, this PR #6047 should fix it

@phil-davis
Copy link
Contributor

PR #6047 fixed this, except for unusual timing situations if uploading a large file followed by overwriting with small content.

I raised issue #6081 to improve the solution.

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

Successfully merging a pull request may close this issue.

5 participants