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

ci: add new github macos-14 builders #22461

Closed
wants to merge 1 commit into from
Closed

ci: add new github macos-14 builders #22461

wants to merge 1 commit into from

Conversation

markemer
Copy link
Member

Description

Add the newly available macOS 14 builders to main.yml

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 14

Verification

Have you

@macportsbot macportsbot added the by: member Created by a member with commit rights label Jan 31, 2024
@markemer markemer marked this pull request as draft January 31, 2024 00:53
@markemer
Copy link
Member Author

I converted this to draft because a simple fix is doing this:

  macOS version: 14.2.1
  curl: (22) The requested URL returned error: 404
  IP address: 13.105.117.171
  curl: (22) The requested URL returned error: 404
  curl: (22) The requested URL returned error: 403
  Can reach private packages server: no

@markemer
Copy link
Member Author

Ok, it appears this will depend on https://github.com/macports/macports-ci-files/blob/main/.github/workflows/main.yml being changed too - I'll put up a PR for that.

@pmetzger
Copy link
Member

pmetzger commented Jan 31, 2024

Do we want to keep MacOS 11 in the runners? It is no longer getting security updates etc.

@markemer
Copy link
Member Author

That's a good question. The buildbot has everything going back to the beginning of time, so it might be safer to ditch it.

@ryandesign
Copy link
Contributor

The more systems we can test on in CI, the better, no?

@aeiouaeiouaeiouaeiouaeiouaeiou
Copy link
Contributor

aeiouaeiouaeiouaeiouaeiouaeiou commented Jan 31, 2024

It's better to keep all build machines while official support is ongoing. They use only GitHub infrastructure and don't require payment (unlike arm64 VMs - need them too, but it's quite an expensive piece of cake) actions/runner-images#9254.

@ryandesign
Copy link
Contributor

Oh: https://github.blog/changelog/2024-01-30-github-actions-macos-14-sonoma-is-now-available/

  • the macos-14 runner is arm64
  • the macos-11 runner is now deprecated and will be eliminated by June 2024

@ryandesign
Copy link
Contributor

  macOS version: 14.2.1
  curl: (22) The requested URL returned error: 404
  IP address: 13.105.117.171
  curl: (22) The requested URL returned error: 404
  curl: (22) The requested URL returned error: 403
  Can reach private packages server: no

I need to update the list of allowed IP address ranges on the private server, and automate that while I'm at it.

@markemer
Copy link
Member Author

markemer commented Jan 31, 2024

Keeping in draft until it works - which I think is just waiting on IP changes?

Also, I assume we would like to fix this warning:

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

by updating to actions/upload-artifact@v4 but I'll do that under a separate PR unless people think I should include it here.

@ryandesign
Copy link
Contributor

waiting on IP changes?

The new IP address ranges are already in use by the existing runners so fixing that needn't prevent this from being merged.

Until I fix the IP allow list on the private server, it'll just mean that the runners have to build any nondistributable ports from source, if they happen to be using one of those new IPs.

The reason why it's not a simple change is that I had been using a few allow rules for the IP addresses I had observed being in use before, but now that I want to use the official GitHub API for determining their IP addresses I see that there are over 3,500 IP address ranges used for Actions runners and I don't think it is designed to accommodate that many allow rules. Even if I limit it to the 450 or so IP address ranges that I suspect are the ones used for macOS runners, that's still a lot and I want to switch to a different method.

@markemer
Copy link
Member Author

markemer commented Feb 2, 2024

Oh, ok - so that's not the cause of the failure maybe then? I'll play around.

@markemer
Copy link
Member Author

markemer commented Feb 2, 2024

I figured it out - macports-ci-files doesn't have files for major number 23 - that's the 404s - but they're async so they threw me off.

@jmroot I can't push the button on those actions, can you?

And I can update bootstrap.sh to 2.9.1 so 2.9.1-23 will be available for download.

@markemer markemer requested a review from jmroot February 2, 2024 05:06
- remove deprecated macos-11 builder
- add new arm64 macos-14 builder
- change MACPORTS_VERSION to 2.9.1 so job will succeed
@jmroot
Copy link
Member

jmroot commented Feb 2, 2024

@markemer
Copy link
Member Author

markemer commented Feb 2, 2024

I was going to update the CI config next week, as per https://github.com/macports/macports-base/blob/master/portmgr/ReleaseProcess.md#use-of-new-features-in-portfiles

This could wait - should just be ready to go then. I could also fix it for 2.9.0 major 23 but it doesn't seem like it's worth it.

@jmroot jmroot closed this in d3e8fdf Feb 7, 2024
@markemer markemer deleted the add-macos-14 branch February 9, 2024 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
by: member Created by a member with commit rights
Development

Successfully merging this pull request may close these issues.

6 participants