Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add race condition checking to CI #491

Closed
anoadragon453 opened this issue Jun 11, 2018 · 0 comments · Fixed by #2814
Closed

Add race condition checking to CI #491

anoadragon453 opened this issue Jun 11, 2018 · 0 comments · Fixed by #2814
Assignees
@anoadragon453
Labels
tests Issues related to tests. Missing/Flakey/etc
Milestone
#1 (Client-Side) ...

Comments

@anoadragon453
Copy link
Member

Go has a built-in race detector, which gb can utilize with the -race flag.

May be worthwhile to include as part of our CI.

https://blog.golang.org/race-detector
@anoadragon453 anoadragon453 changed the title Add race condition checking into travis Add race condition checking to CI Jul 17, 2019
@anoadragon453 anoadragon453 mentioned this issue Jul 17, 2019
Closed
@anoadragon453 anoadragon453 self-assigned this Jul 17, 2019
@kegsay kegsay added the tests Issues related to tests. Missing/Flakey/etc label Apr 14, 2020
@S7evinK S7evinK mentioned this issue Oct 20, 2022
Merged
S7evinK added a commit that referenced this issue Oct 27, 2022
@S7evinK
Add scheduled tasks to run tests with race detection (#2814)
fa96811 
Needs matrix-org/sytest#1308 to be actually
useful.
Not sure if we need to run Sytest in all combinations with enabled race
detection.

Closes #491
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@anoadragon453 anoadragon453

Labels
tests Issues related to tests. Missing/Flakey/etc
Projects
None yet
Milestone
#1 (Client-Side) Bot Hosting
2 participants
@anoadragon453 @kegsay