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

Arbitrary MSYS2 Python failures on Windows builds #6523

Closed
3 of 11 tasks
vintagepc opened this issue Nov 6, 2022 · 8 comments
Closed
3 of 11 tasks

Arbitrary MSYS2 Python failures on Windows builds #6523

vintagepc opened this issue Nov 6, 2022 · 8 comments
Assignees
Labels
Area: Python bug report investigate Collect additional information, like space on disk, other tool incompatibilities etc. OS: Windows

Comments

@vintagepc
Copy link

Description

I use MSYS2 to build Windows binaries of QEMU in my actions. Currently, I'm getting arbitrary python failures during the build process - sometimes the build works, and sometimes it fails despite there being no changes in the affected region.

Here is an example of a failed build:
https://github.com/vintagepc/MINI404/actions/runs/3404920110/jobs/5662636923#step:1:2

And this build completed successfully:
https://github.com/vintagepc/MINI404/actions/runs/3404850675/jobs/5662389192

Both builds ran within a few minutes of each other.

There were no differences to the build configuration or code between the two except to fix a warning (in a file that isn't even compiled until much, much later) The build.yml change is for a different job entirely.
vintagepc/MINI404@1fc2742

Platforms affected

  • Azure DevOps
  • GitHub Actions - Standard Runners
  • GitHub Actions - Larger Runners

Runner images affected

  • Ubuntu 18.04
  • Ubuntu 20.04
  • Ubuntu 22.04
  • macOS 10.15
  • macOS 11
  • macOS 12
  • Windows Server 2019
  • Windows Server 2022

Image version and build link

Build logs linked above.

Current runner version: '2.298.2'
Operating System
Microsoft Windows Server 2022
10.0.20348
Datacenter
Runner Image
Image: windows-2022
Version: 20221027.1

Is it regression?

OS 10.0.20348/ Runner image 20221019.1

Expected behavior

Build to complete successfully

Actual behavior

Python command to generate keymaps seems to be arbitrarily failing or passing with no detailed error. This area of the code hasn't been changed in months and comes directly from upstream QEMU.

https://github.com/vintagepc/MINI404/actions/runs/3381838302/jobs/5624470117 is an example of a build with a python failue elsewhere in the process, also in code that hasn't been altered.

Repro steps

See build_msys section in this workflow: https://github.com/vintagepc/MINI404/blob/MINI404/.github/workflows/build.yml

Summarily:

  1. Install MSYS and required packages
  2. Checkout MINI404 repository with autoCRLF false
  3. configure and build per workflow commands.
@al-cheb al-cheb added Area: Python OS: Windows investigate Collect additional information, like space on disk, other tool incompatibilities etc. and removed needs triage labels Nov 7, 2022
@al-cheb
Copy link
Contributor

al-cheb commented Nov 7, 2022

Hey @vintagepc.
We will take a look at it.

@ddobranic ddobranic self-assigned this Nov 7, 2022
@ddobranic
Copy link
Contributor

hello @vintagepc! in successful build log i can see checkout step for 114-fix-workflows differs from checkout step from unsuccessful build

@vintagepc
Copy link
Author

vintagepc commented Nov 7, 2022

Yes, that is the diff linked and noted in the initial report - however none of these changes impact this region of the code.

I should add I have some private forks of this repository that have also started experiencing similar failures during the msys build.

@eli-schwartz
Copy link

Yeah, there seems to be a general issue with msys2 python crashing unpredictably for no obvious reason, just in GitHub Actions. It's definitely not a problem with the projects running the workflows, because... restarting the workflow with the same context causes it to pass.

@vintagepc
Copy link
Author

That's some interesting reading. Glad to hear it's not just me and others are also experiencing this problem in GA

@ddobranic
Copy link
Contributor

@vintagepc do you still have the same issue? maybe using the previous version of msys2 could help? please check example here

@vintagepc
Copy link
Author

vintagepc commented Nov 17, 2022

As far as I know the problem still exists, but I was able to work around it by force-downgrading to an older MSYS2 python package. However, this is not a permanent fix, as MSYS doesn't support this and the old packages might be removed or become incompatible at any time.

Note my action still uses setup-msys@v2, not the v2.5 mentioned in the linked issue.

@ddobranic
Copy link
Contributor

@vintagepc since we cannot do more for now, i will close the issue. please feel free to reopen if you have any additional questions or concerns.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Python bug report investigate Collect additional information, like space on disk, other tool incompatibilities etc. OS: Windows
Projects
None yet
Development

No branches or pull requests

4 participants