[SERVER-42468] TransactionCoordinatorMetricsTest has race between transaction coordinator's lifetime and checking for log lines Created: 29/Jul/19  Updated: 29/Oct/23  Resolved: 29/Jul/19

Status: Closed
Project: Core Server
Component/s: Sharding
Affects Version/s: None
Fix Version/s: 4.2.0-rc5, 4.3.1

Type: Bug Priority: Major - P3
Reporter: Kaloian Manassiev Assignee: Kaloian Manassiev
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Depends
Backwards Compatibility: Fully Compatible
Operating System: ALL
Backport Requested:
v4.2
Sprint: Sharding 2019-08-12
Participants:
Linked BF Score: 22

 Description   

There is a race condition in the TransactionCoordinatorMetricsTests between when we stop capturing the test's log messages stream and the completion of the asynchronous completion of the transaction coordinator, which might not have gotten to log the slow transaction message by that time.

In order to fix it, we should wait for the onCompletion notification on the TransactionCoordinator to fire before we stop recording the log messages.



 Comments   
Comment by Githook User [ 29/Jul/19 ]

Author:

{'name': 'Kaloian Manassiev', 'email': 'kaloian.manassiev@mongodb.com', 'username': 'kaloianm'}

Message: SERVER-42468 Always wait for the TransactionCoordinator to complete in the Metrics unit-tests

(cherry picked from commit 25242cc39477596e23ae0ab4d4fd21b63a2f3ed6)
Branch: v4.2
https://github.com/mongodb/mongo/commit/e6da060a704bd7c7eb62444ec47572c76a7b5e75

Comment by Githook User [ 29/Jul/19 ]

Author:

{'name': 'Kaloian Manassiev', 'email': 'kaloian.manassiev@mongodb.com', 'username': 'kaloianm'}

Message: SERVER-42468 Always wait for the TransactionCoordinator to complete in the Metrics unit-tests
Branch: master
https://github.com/mongodb/mongo/commit/25242cc39477596e23ae0ab4d4fd21b63a2f3ed6

Generated at Thu Feb 08 05:00:34 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.