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

fix(test): fix failed test cases due to race condition #105

Merged
merged 5 commits into from
Jul 3, 2024

Conversation

derekbit
Copy link
Member

@derekbit derekbit commented Jul 2, 2024

Which issue(s) this PR fixes:

Issue longhorn/longhorn#8880

What this PR does / why we need it:

Special notes for your reviewer:

Additional documentation or context

@derekbit derekbit self-assigned this Jul 2, 2024
@derekbit derekbit changed the title fix(test): fix failed test cases due to reace condition fix(test): fix failed test cases due to race condition Jul 2, 2024
@derekbit derekbit force-pushed the dev branch 2 times, most recently from 8e3047d to a440021 Compare July 2, 2024 16:15
@derekbit
Copy link
Member Author

derekbit commented Jul 2, 2024

@FrankYang0529 The CI is queued and stuck. Can you help check it? Thanks.
https://github.com/longhorn/sparse-tools/actions/runs/9764492417/job/26953145525

Start working after ten minutes..

Copy link

codecov bot commented Jul 2, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Longhorn 8880

Signed-off-by: Derek Su <[email protected]>
@derekbit derekbit merged commit 92451e3 into longhorn:master Jul 3, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants