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 instability #2971

Merged
merged 1 commit into from
Aug 9, 2024
Merged

Fix test instability #2971

merged 1 commit into from
Aug 9, 2024

Conversation

keegan-caruso
Copy link
Contributor

@keegan-caruso keegan-caruso commented Aug 9, 2024

Revert some of the test changes introduces in #2950

That change introduced test instability

fixes #2964

The changes introduced test instability
@keegan-caruso keegan-caruso requested a review from a team as a code owner August 9, 2024 02:25
Copy link
Collaborator

@jennyf19 jennyf19 left a comment

Choose a reason for hiding this comment

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

@keegan-caruso keegan-caruso merged commit ecd28dd into master Aug 9, 2024
5 checks passed
@keegan-caruso keegan-caruso deleted the kecaruso/fix-unstable-tests branch August 9, 2024 02:36
@keegan-caruso keegan-caruso mentioned this pull request Aug 9, 2024
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.

CacheExtensions_CcaAlreadyExistsL2_TestsAsync is flaky
3 participants