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

the race conditions in tests #7847

Merged
merged 1 commit into from
Jan 4, 2024
Merged

the race conditions in tests #7847

merged 1 commit into from
Jan 4, 2024

Conversation

2403905
Copy link
Contributor

@2403905 2403905 commented Nov 29, 2023

Description

Fix the race conditions in tests

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@butonic
Copy link
Member

butonic commented Nov 29, 2023

Just to let you know, we ran into random freezes in ci runs because the ociswrapper would not flush the stdout buffer: #7843

Maybe rebase this pr to make sure to get those sideeffects out of the way.

Copy link

sonarqubecloud bot commented Jan 4, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@2403905 2403905 merged commit 82d80d8 into owncloud:master Jan 4, 2024
4 checks passed
ownclouders pushed a commit that referenced this pull request Jan 4, 2024
the race conditions in tests
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.

The race conditions in tests
3 participants