-
Notifications
You must be signed in to change notification settings - Fork 167
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
Add Apple silicon macs #2383
Add Apple silicon macs #2383
Conversation
Manual intervention needed so far: Brew needs a workround patch untill upstream is patched: Homebrew/brew#7857 |
Failures: installing tap2junit:
|
Looks like for some reason |
oo good spot, for some reason these brand new macs released this year still come with python 2 as the system python @richardlau is there a way to set which python ansible uses on the target machine? |
Yes, see e.g. Line 42 in 022ee21
|
That's certainly odd -- I'm sure we've had lots of reports over in node-gyp where the system python on newer macOS is Python 3. |
from updating the python used Its now pointing out that this is the wrong link:
On the DTK pip is under in other news I get a clean run through of the playbooks now |
so |
Second machine setup, have discovered the
|
Sorry, missed the request to add IP's looking at it now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Should be active now |
first run through of new job https://ci.nodejs.org/job/node-test-commit-osx-arm/nodes=osx11/1/console |
@AshCripps seems like the changes will affect all the Macs not just the new ones, can you confirm you've already validated they will all work ok on the old machines as well? |
@mhdawson haven't tested yet will do that now. I tried to spawn a fresh instance in orka to save affecting one of the test machines but do to my current internet issues I can't load the VPN so will have to use one of the test machines. |
57c28e2
to
1dd27f9
Compare
Rebased and pushed to fix merge conflicts |
This pr will add the DTKs kindly donated to us by macstadium and any edits needed to playbooks.
refs: nodejs/TSC#886
@nodejs/build-infra can the ips be added to the test jenkins firewall please.