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.

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

Skip flakey tests in CI #50071

Merged
merged 2 commits into from
Jun 7, 2023
Merged

Skip flakey tests in CI #50071

merged 2 commits into from
Jun 7, 2023

Conversation

LilithHafner
Copy link
Member

@LilithHafner LilithHafner commented Jun 5, 2023

This PR marks tests that have been failing CI recently as skipped so that folks submitting unrelated PRs don't have to go through test logs to make sure that failures are unrelated (and to reduce the likelihood and frequency of this kind of mistake)

Addmul
Failure on linux 0c774c7: buildkite
Failure on windows c3d70e8: buildkite
Failure on macos 94bc2f3: buildkite
Reported: #49966

Spawn and wait lots of tasks
Failure on mac f407a4c: buildkite
Reported: #43124 (comment)

cmdlineargs with --threads
Failure on mac 9b27a8f8a5: buildkite
Failure on mac 20752db7c1: buildkite
Failure on linux d912d85c42: buildkite
Failure on linux 318f0ead95: buildkite
Failure on FreeBSD: #49072 (comment)
Reported: #49154

@LilithHafner LilithHafner added the domain:ci Continuous integration label Jun 5, 2023
@@ -189,7 +190,8 @@ end

returned_mat = mul!(C, Af, Bf, α, β)
@test returned_mat === C
@test collect(returned_mat) ≈ α * Ac * Bc + β * Cc rtol=rtol
# This test is skipped because it is flakey, but should be fixed and put back (see #49966)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah! I thought I broke it in #49405 (comment)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

haha, I don't think so

withenv("JULIA_NUM_THREADS" => nt) do
@test read(`$exename --threads=2 -e $code`, String) ==
read(`$exename -t 2 -e $code`, String) == "2"
if Sys.iswindows()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not entirely sure Windows is immune from issues, maybe it just happens less frequently, I seem to remember @vtjnash saying that these tests are a bit unreliable in general (do I remember wrong?). But it'd also be sad to completely miss this block of tests.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It hasn't failed for Windows in the past 40 commits to master so I'm hopeful, but far from confident that these are reliable on Windows. Not wanting to miss this entire block also motivated my not skipping these on Windows.

Copy link
Sponsor Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think Windows does not have this segfault issue, but because of that, it might hang instead.

@LilithHafner
Copy link
Member Author

Do we know why windows tests have "Received cancellation signal, interrupting" and fail after 1s?

@LilithHafner LilithHafner merged commit 13e40af into master Jun 7, 2023
@LilithHafner LilithHafner deleted the skip-flakey-tests branch June 7, 2023 14:01
LilithHafner added a commit that referenced this pull request Sep 14, 2024
Partial revert of #50071, inspired by conversation in #49966 (comment)
LilithHafner added a commit that referenced this pull request Sep 15, 2024
Partial revert of #50071, inspired by conversation in
#49966 (comment)

Ran the tests 100 times to make sure we're not putting back
something that's still flaky.

Closes #49966
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:ci Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants