Skip to content

gh-159612: Remove macOS x86_64 from JIT CI due to LLVM timeouts - #160033

Merged
hugovk merged 1 commit into
python:mainfrom
hugovk:3.16-rm-jit-macos-intel
Oct 26, 2026
Merged

hugovk merged 1 commit into
python:mainfrom
hugovk:3.16-rm-jit-macos-intel

Conversation

@hugovk

@hugovk hugovk commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Homebrew 7 no longer provides binary "bottles" for macOS Intel:

https://brew.sh/2026/09/13/homebrew-7.0.0/#-macos-users

Which means LLVM has to build from source on this platform. This takes more than the 60-minute limit and times out.

PEP 836 has been withdrawn, and in any case only targetted tier 1, where macOS Intel is tier 2.

Let's remove macOS Intel from the JIT CI. If any future work wants to test it, let's figure it out then.

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What about the tail-call workflow, it's also affected?

@ned-deily

ned-deily commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

What about the tail-call workflow, it's also affected?

There should be no need to use a non-Apple clang for tail-call builds these days. The only need in our installer builds is for creating the JIT stencils.

@StanFromIreland

StanFromIreland commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

So we can switch to the Xcode Clang in that job IIUC?

@ned-deily

Copy link
Copy Markdown
Member

Unless there's some other gotcha, yes. Let's try it with just Xcode clang.

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. We have an Intel macOS buildbot: https://buildbot.python.org/#/workers/68, "x86-64 MacOS Intel NoGIL 3.x" and "x86-64 MacOS Intel ASAN NoGIL 3.x".

@itamaro

itamaro commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

considering darwin-intel is tier-2, we could just drop all CI coverage for that platform, and keep just the buildbots

@hugovk
hugovk merged commit c436e8d into python:main Oct 26, 2026
83 checks passed
@hugovk
hugovk deleted the 3.16-rm-jit-macos-intel branch October 26, 2026 04:49
@miss-islington-app

Copy link
Copy Markdown

Thanks @hugovk for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14, 3.15.
🐍🍒⛏🤖

@bedevere-app

bedevere-app Bot commented Oct 26, 2026

Copy link
Copy Markdown

GH-160694 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 bugs and security fixes label Oct 26, 2026
@bedevere-app

bedevere-app Bot commented Oct 26, 2026

Copy link
Copy Markdown

GH-160695 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Oct 26, 2026
hugovk added a commit that referenced this pull request Oct 26, 2026
GH-160033) (#160695)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
hugovk added a commit that referenced this pull request Oct 26, 2026
GH-160033) (#160694)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@StanFromIreland

Copy link
Copy Markdown
Member

What about the other half? This only fixes part of the issue, the TC jobs still time out?

@hugovk

hugovk commented Oct 26, 2026

Copy link
Copy Markdown
Member Author

This PR was for the JIT half, would you like to do the TC half?

@StanFromIreland

Copy link
Copy Markdown
Member

I don't mind, I'd just like it to be fixed, as it has troubled me on a few PRs recently.

@StanFromIreland

Copy link
Copy Markdown
Member

I wrote #160713.

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

Labels

infra CI, GitHub Actions, buildbots, Dependabot, etc. OS-mac skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants