Skip to content

Conversation

@marcelosalloum
Copy link
Contributor

@marcelosalloum marcelosalloum commented Nov 4, 2025

What

Upgrade workflows to use macos-15-* rather than macos-13

Why

  1. It's about to be deprecated: announcement 🔗
  2. It's already making the CI fail: example run 🔗

Notes on Runner Compatibility

After extensive testing across macOS runner types:

Runner Result Notes
macos-15-large Passes Stable with full setup.
macos-14 Passes Works consistently.
🟠 macos-15-intel ⚠️ Intermittent Suitable for lighter setups but fails under heavier load — -large is usually more stable.
🟠 macos-15 ⚠️ It depends Does not support nested virtualization (required by Lima/QEMU), so cannot be used in some setups.

⚠️ Disabled Job

I couldn't make the ledger-emulator.yml workflow succeed for macos despite many attempts, so it was temporarily disabled, to be re-added in #2299.

@marcelosalloum marcelosalloum requested a review from a team November 4, 2025 21:24
@marcelosalloum marcelosalloum self-assigned this Nov 4, 2025
@github-project-automation github-project-automation bot moved this to Backlog (Not Ready) in DevX Nov 4, 2025
Copy link
Member

@leighmcculloch leighmcculloch left a comment

Choose a reason for hiding this comment

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

Few comments. Some are out of scope of this PR but we should follow up.

Copy link
Member

@leighmcculloch leighmcculloch left a comment

Choose a reason for hiding this comment

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

Lgtm, although you might want to test the macos and arm builds on the pull request temporarily before merging and finding that they might fail on the push event.

…latest`, and from `ubuntu-jammy-8-cores-arm64` to `ubuntu-24.04-arm`"

This reverts commit 79ecbc7.
@marcelosalloum marcelosalloum changed the title CI: upgrade workflows to use macos-15 rather than macos-13 CI: upgrade workflows to use macos-15(-intel) rather than macos-13 Nov 4, 2025
@marcelosalloum marcelosalloum moved this from Backlog (Not Ready) to Needs Review in DevX Nov 4, 2025
since it offers better support for nested
virtualization. This change
is made for the bindings-ts.yml, ledger-emulator.yml, and rpc-tests.yml
workflows.
…able the respective linux counterparts. The goal is to test the macos runner
@marcelosalloum marcelosalloum changed the title CI: upgrade workflows to use macos-15(-intel) rather than macos-13 CI: upgrade workflows to use macos-15(*) rather than macos-13 Nov 5, 2025
@marcelosalloum marcelosalloum force-pushed the ci/upgrade-mac-ci-version branch from 35f4db4 to ea04e71 Compare November 6, 2025 19:31
@marcelosalloum marcelosalloum enabled auto-merge (squash) November 19, 2025 18:01
@marcelosalloum marcelosalloum merged commit 1e12cb8 into main Nov 19, 2025
31 checks passed
@marcelosalloum marcelosalloum deleted the ci/upgrade-mac-ci-version branch November 19, 2025 18:09
@github-project-automation github-project-automation bot moved this from Needs Review to Done in DevX Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants