Skip to content

fix: owncloud test sometimes fail#2684

Merged
Xuanwo merged 1 commit intoapache:mainfrom
dqhl76:fix-ci-owncloud
Jul 21, 2023
Merged

fix: owncloud test sometimes fail#2684
Xuanwo merged 1 commit intoapache:mainfrom
dqhl76:fix-ci-owncloud

Conversation

@dqhl76
Copy link
Copy Markdown
Member

@dqhl76 dqhl76 commented Jul 21, 2023

Summary

Our integration test for owncloud has chance to fail

We try to solve this by reducing concurrency #2667

but it fail again, so this PR try to add redis to make owncloud happy

I found a related discussion https://central.owncloud.org/t/file-is-locked-how-to-unlock/985.
Maybe we can try to use redis to reduce the unexpected lock happen?

https://github.com/apache/incubator-opendal/actions/runs/5606271748/jobs/10256268444

Signed-off-by: dqhl76 <dqhl76@gmail.com>
@dqhl76 dqhl76 requested review from PsiACE and Xuanwo as code owners July 21, 2023 08:35
@github-actions github-actions Bot added the releases-note/fix The PR fixes a bug or has a title that begins with "fix" label Jul 21, 2023
@dqhl76
Copy link
Copy Markdown
Member Author

dqhl76 commented Jul 21, 2023

image I rerun the action for 5 times. It seems has been fixed this time.

Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@Xuanwo Xuanwo merged commit 5edd772 into apache:main Jul 21, 2023
@dqhl76 dqhl76 deleted the fix-ci-owncloud branch July 21, 2023 08:45
@suyanhanx
Copy link
Copy Markdown
Member

Cool!

@oowl oowl mentioned this pull request Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/fix The PR fixes a bug or has a title that begins with "fix"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants