-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
make TSB conformant with respect to multiple parallel operations #18537
make TSB conformant with respect to multiple parallel operations #18537
Conversation
3227ee4
to
c1e12db
Compare
worth adding a test that does multiple binds/unbinds concurrently for a few iterations? I know we can't guarantee we hit the conflict but it would be a nice smoke test. |
(otherwise lgtm. should be backported to ose:enterprise-3.7 and ose:enteprise-3.8) |
082e04a
to
02e836f
Compare
tests added, thanks |
02e836f
to
4e1a952
Compare
provision test? |
4e1a952
to
1d7fd4f
Compare
provision test added. |
/lgtm |
1d7fd4f
to
27f2429
Compare
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bparees, jim-minter The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
Automatic merge from submit-queue (batch tested with PRs 18487, 18537, 18630). |
No description provided.