Skip to content

Fix Windows Gitlab test running#2870

Merged
henryiii merged 4 commits into
mainfrom
gitlab-ci-debug
Jun 1, 2026
Merged

Fix Windows Gitlab test running#2870
henryiii merged 4 commits into
mainfrom
gitlab-ci-debug

Conversation

@joerick
Copy link
Copy Markdown
Contributor

@joerick joerick commented May 26, 2026

These have been failing on main for a little while.

Don't look like a big issue. py is trying to execute the shebang in run_tests.py, which is a posix thing. Trying to use python directly...

@henryiii
Copy link
Copy Markdown
Contributor

henryiii commented May 26, 2026

The python launcher for Windows, py, is supposed to be able to read shebangs on Windows and respect common patterns. Ahh, I see, the uv part is probably confusing it.

@joerick joerick marked this pull request as ready for review May 26, 2026 14:42
@joerick joerick marked this pull request as draft May 26, 2026 16:48
@henryiii henryiii marked this pull request as ready for review May 31, 2026 22:02
joerick and others added 4 commits May 31, 2026 18:02
- Windows: remove stale servercore:1809 image and choco install
  (runners are now Windows 2022 VMs with Python 3.13 pre-installed)
- macOS: update deprecated macos-14-xcode-15 to macos-15-xcode-16
- Linux: bump Python image from 3.12 to 3.13
- All jobs: also run on gitlab* branches with CIBW_ENABLE=all

Generated with the help of AI (Amp/Claude Opus 4.7)

Amp-Thread-ID: https://ampcode.com/threads/T-019e6496-f89a-74e7-9f6b-655574290e68
Co-authored-by: Amp <amp@ampcode.com>
GraalPy is JVM-based and slow to start; on the small (2-core) GitLab
SaaS Windows runners, virtualenv's interpreter query for graalpy.exe
times out, failing test_0_basic::test[pip]. Restrict the Windows job's
CIBW_ENABLE to all groups except graalpy (other platforms keep "all").

CIBW_ENABLE rather than CIBW_SKIP is used so the test's expected-wheel
set (derived from CIBW_ENABLE) stays consistent with what is built.

Assisted-by: ClaudeCode:claude-opus-4.8
Signed-off-by: Henry Schreiner <henryfs@princeton.edu>
@henryiii henryiii merged commit 52f77ea into main Jun 1, 2026
46 checks passed
@henryiii henryiii deleted the gitlab-ci-debug branch June 1, 2026 03:23
@joerick
Copy link
Copy Markdown
Contributor Author

joerick commented Jun 3, 2026

Thanks for finishing this off, henryiii! I had the same suspicion about the startup time as you, but I never managed to verify it. Did you manage to prove that's the problem? Either way, it's probably a bit of an edge case.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants