-
Notifications
You must be signed in to change notification settings - Fork 56
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
Windows CI still flaky (again) #252
Comments
I was unable to reproduce this locally over 25 runs. I wonder if there is some sort of parallelism issue here. |
I think this Issue may be closed |
Fingers crossed |
Seems Windows CI is still flaky :( |
Can you give me a link? @kyouko-taiga |
Seems I can't get a link to the failed CI jobs I re-ran. I'll post a link here next time there's a Windows failure. |
I've been looking at that history but couldn't find the failure. I don't remember exactly which branch it was, I commit a lot of stuff these days. The failure was about 3 days ago, feel free to look for yourself. |
https://github.com/val-lang/val/actions/runs/4242066814/jobs/7373093947 |
Here's another one: https://github.com/val-lang/val/actions/runs/4272221910/attempts/1 |
Sadly, the flakiness is getting worse and starts seriously impeding development. So I decided to disable Windows CI until we can figure out a solution (see #495). |
I noticed that the things happens when using Swift locally same as CI. |
I'm thinking this isn't a problem we can solve either, so what should we do? |
I have an idea |
Assign this issue to me? |
@C-BJ would you be able to get a stack trace or minidump from the failure? That would help understand what is going on (hitting an assert/invalid memory access). |
@compnerd how? I just run a Swift command and it doesn't respond, and then it exits |
#517 Need to be solved first |
If it crashes, you should be able to enable user mode dumps and collect a minidump. There are details on how to configure that at https://learn.microsoft.com/en-us/windows/win32/wer/collecting-user-mode-dumps. |
Is this thread relevant to our issue: https://github.com/orgs/community/discussions/26933 |
At this stage we have temporarily solved this problem by retrying. #521 |
I've recently reworked the CI quite substantially and uncovered/suppressed one possible cause of this problem so I'm going to close this issue until such a time as we see problems again. At that point we should file a new issue. |
Well no, that wasn't the problem. Reopening until I implement the "try again" workaround. |
I implemented a lot of workarounds, and now we're all green again… for now. I only pray this is an SPM problem that will be fixed by using cmake. |
https://github.com/val-lang/val/actions/runs/3990232735/attempts/1
The text was updated successfully, but these errors were encountered: