-
Notifications
You must be signed in to change notification settings - Fork 68
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
Active active test fix #869
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you for all these fixes to make it more stable!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @ryanemerson!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good spot @mhajas, I've accepted your suggestion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @ryanemerson
Signed-off-by: Ryan Emerson <[email protected]>
Signed-off-by: Ryan Emerson <[email protected]>
…bhook is triggered when executing tests in close succession Signed-off-by: Ryan Emerson <[email protected]>
…ave database in an inconsistent state" This reverts commit 92cf75b. Signed-off-by: Ryan Emerson <[email protected]>
Signed-off-by: Ryan Emerson <[email protected]>
Signed-off-by: Ryan Emerson <[email protected]>
Signed-off-by: Ryan Emerson <[email protected]>
Signed-off-by: Ryan Emerson <[email protected]>
Co-authored-by: Michal Hajas <[email protected]> Signed-off-by: Ryan Emerson <[email protected]>
2b00beb
to
eabd21f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ryanemerson!
Closes #854
The source of the original failures was fixed by keycloak/keycloak#30841
However, when investigating the issue I realised that the testsuite was very flaky and failures often occurred when executing the
FailoverTest
in conjunction with the other tests. This PR reenables the active-active tests as well adding several fixes/improvements to the testsuite to reduce the chance of flaky failures.