Skip to content

Conversation

MarcelKoch
Copy link
Member

This PR disables the threadsanitizer CI job temporarily. In its current state, the threadsanitizer CI job has the following issues:

  • it's slow. The run can take >10h. Running it locally is much faster, so some configuration might be wrong.
  • false positives. Many tests fail, with either broken stack traces, or non-reproducible output.

@MarcelKoch MarcelKoch added this to the Ginkgo 1.9.0 milestone Nov 27, 2024
@MarcelKoch MarcelKoch requested a review from a team November 27, 2024 14:47
@MarcelKoch MarcelKoch self-assigned this Nov 27, 2024
@ginkgo-bot ginkgo-bot added the reg:ci-cd This is related to the continuous integration system. label Nov 27, 2024
@MarcelKoch MarcelKoch added the 1:ST:ready-to-merge This PR is ready to merge. label Nov 28, 2024
@MarcelKoch MarcelKoch force-pushed the disable-threadsanitizer branch from fedede5 to aa03ce4 Compare November 28, 2024 07:47
In its current state the threadsanitizer CI job has the following issues:
- it's slow. The run can take >10h. Running it locally is much faster, so some configuration might be wrong.
- false positives. Many tests fail, with either broken stack traces, or non locally reproducible output.
@MarcelKoch MarcelKoch force-pushed the disable-threadsanitizer branch from aa03ce4 to 4b4f1b6 Compare November 28, 2024 12:49
@MarcelKoch MarcelKoch merged commit f95e1ff into develop Nov 28, 2024
6 of 11 checks passed
@MarcelKoch MarcelKoch deleted the disable-threadsanitizer branch November 28, 2024 12:50
@ginkgo-bot
Copy link
Member

Error: PR already merged!

Copy link

MarcelKoch added a commit to MarcelKoch/ginkgo that referenced this pull request Dec 2, 2024
This PR disables the threadsanitizer CI job temporarily. In its current state, the threadsanitizer CI job has the following issues:
- it's slow. The run can take >10h. Running it locally is much faster, so some configuration might be wrong.
- false positives. Many tests fail, with either broken stack traces, or non-reproducible output.

Related PR: ginkgo-project#1735
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1:ST:ready-to-merge This PR is ready to merge. reg:ci-cd This is related to the continuous integration system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants