This ticket is filed in response to BF-14376. The test assertion, and the code that removes the coordinator from the catalog are not synchronized, and could lead to the an execution where the assertion runs before the coordinator is removed.
–
The CoordinatorsRemoveThemselvesFromCatalogWhenTheyComplete test waits on the future for the coordinator to complete and then checks if it is removed from the catalog. There is no coordination between the code that removes the coordinator from the catalog, which is an async callback that executes after the onCompletion future is ready, and the test assertion. Therefore, there are no ordering guarantees between the removal callback and the assertion code, which is what we see as the result of this build failure.