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/integration: run parallel tests concurrently with serial tests #808
Conversation
Codecov Report
@@ Coverage Diff @@
## main #808 +/- ##
=======================================
Coverage 79.15% 79.15%
=======================================
Files 128 128
Lines 8768 8768
=======================================
Hits 6940 6940
Misses 1604 1604
Partials 224 224 Continue to review full report at Codecov.
|
fc046eb
to
8df8379
Compare
8df8379
to
7c9680f
Compare
4e7a6fb
to
8fe60c8
Compare
test/integration/main_test.go
Outdated
for _, test := range tests { | ||
test := test | ||
|
||
if strings.Contains(test.Name, "Parallel") { |
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.
@enj It looks like you meant to say "ends with _Parallel
" here, so we don't accidentally classify TestMyParallelFunctionality
as one of these special tests.
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.
Or maybe "ends with _Parallel
OR contains _Parallel/
"?
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.
Oh I was being a bit lazy here, I can change it to "ends with _Parallel
"
Signed-off-by: Monis Khan <mok@vmware.com>
Instead of blindly waiting long enough for a disruptive change to have been observed by the old leader and followers, we instead rely on the approximation that checkOnlyLeaderCanWrite provides - i.e. only a single actor believes they are the leader. This does not account for clients that were in the followers list before and after the disruptive change, but it serves as a reasonable approximation. Signed-off-by: Monis Khan <mok@vmware.com>
9cc90a3
to
e2cf9f6
Compare
This was broken recently by the improvements in #808.
Signed-off-by: Monis Khan mok@vmware.com
Fixes #803
Fixes #351
Release note: