-
Notifications
You must be signed in to change notification settings - Fork 558
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
test(qa/integration-tests): improve test stability #4692
Conversation
* open the log reader first to avoid concurrent access to the log (e.g. during log compaction)
I ran it multiple times in the CI and it looks good so far 🤞 In general, I'm not so happy about how this test is written because it has some assumptions about how Zeeebe works internally (e.g. snapshotting, log segment directories, etc.). |
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.
👍
bors r+ |
4692: test(qa/integration-tests): improve test stability r=deepthidevaki a=saig0 ## Description * open the log reader first to avoid concurrent access to the log (e.g. during log compaction) ## Related issues closes #4430 # Co-authored-by: Philipp Ossler <philipp.ossler@gmail.com>
Build failed |
bors retry |
Build succeeded |
+ add test case when both INCIDENT CREATED and INCIDENT RESOLVED are processed together closes #4692
Description
Related issues
closes #4430
Pull Request Checklist
mvn clean install -DskipTests
locally before committing