-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Fixed test_profile_max_sessions_for_user test flakiness #52897
Conversation
This is an automated comment for commit 288e3f7 with description of existing statuses. It's updated for the latest CI running
|
Thank you for the quick fix! |
Integration tests (tsan) [2/6] - |
@alexey-milovidov, Mergeable Check doesn't allow to merge this PR, and I cannot update the branch to merge it forcefully |
The test relied on the impossibility of parallel test start and sleeps in queri, which could lead to flackiness. Now the test is tied to the presence of an error in the log and does not use sleep in the query.
Also removed the unnecessary test_profile_max_sessions_for_user_end_session
Changelog category (leave one):
Not for changelog (changelog entry is not required)