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

[v12.x] build: pin macOS GitHub runner to macos-10.15 #41124

Merged

Conversation

richardlau
Copy link
Member

@richardlau richardlau commented Dec 9, 2021

Node.js 12.x requires Python 2 to build. macos-latest GitHub
hosted runners are now macOS 11 and do not have Python 2 available.

Refs: actions/runner-images#4060


Node.js 12 is in maintenance and goes End-of-Life next April (2022). This pull request is to attempt to fix the GitHub actions macOS workflow by pinning the runner to the macOS version we were previously testing on before GitHub updated what macos-latest points to,

Example of failing run on b13340e (current v12.x-staging: https://github.com/nodejs/node/actions/runs/1559350131
image

If this doesn't work I'd propose dropping this workflow from v12.x -- I briefly looked at the Python 3 PRs that landed in the later release lines and I would not be comfortable attempting to backport those to v12.x this late in its lifecycle.

@github-actions github-actions bot added meta Issues and PRs related to the general management of the project. v12.x labels Dec 9, 2021
@richardlau
Copy link
Member Author

This does not affect v14.x and later as they support building with Python 3 and do so in their equivalent macOS GitHub workflows, e.g. for v14.x-staging

PYTHON_VERSION: '3.10'
.

Node.js 12.x requires Python 2 to build. `macos-latest` GitHub
hosted runners are now macOS 11 and do not have Python 2 available.

PR-URL: nodejs#41124
Refs: actions/runner-images#4060
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Beth Griggs <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
@richardlau richardlau force-pushed the v12.x-staging-macos-runner branch from 208c62c to 2d42295 Compare December 13, 2021 13:10
@richardlau richardlau merged commit 2d42295 into nodejs:v12.x-staging Dec 13, 2021
@richardlau
Copy link
Member Author

Landed in 2d42295.

@richardlau richardlau deleted the v12.x-staging-macos-runner branch December 13, 2021 13:11
@richardlau richardlau mentioned this pull request Dec 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta Issues and PRs related to the general management of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants