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] Remove macos from Docker build #1648

Merged
merged 3 commits into from
Oct 24, 2024
Merged

[CI] Remove macos from Docker build #1648

merged 3 commits into from
Oct 24, 2024

Conversation

jiayuasu
Copy link
Member

@jiayuasu jiayuasu commented Oct 16, 2024

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

  • No:
    • this is a CI update. The PR name follows the format [CI] my subject

What changes were proposed in this PR?

  1. macos-12 is being deprecated by GitHub and starts to break sometimes: [macOS] The macOS 12 Actions runner image will begin deprecation on 10/7/24 and will be fully unsupported by 12/3/24 for GitHub and ADO actions/runner-images#10721
  2. Recent QEMU release 9.10 breaks the Colima runtime on both macos and macos 13 (see Docker setup hangs forever on new macos-13 image douglascamata/setup-docker-macos-action#37, and macos-13 not working douglascamata/setup-docker-macos-action#41).

After numerous attempts, the MacOS build sometimes can finish now but still occasionally crash. Therefore, this PR will remove macos build and we plan to add it back in the future.

How was this patch tested?

Passed CI

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the documentation.

@jiayuasu jiayuasu marked this pull request as draft October 21, 2024 20:14
@jiayuasu
Copy link
Member Author

Root cause is the recent QEMU 9.10 breaks with Colima. See another issue here: douglascamata/setup-docker-macos-action#37

brew install qemu.rb 2>&1
continue-on-error: true
- name: Setup tmate session
uses: mxschmitt/action-tmate@v3
Copy link
Member

Choose a reason for hiding this comment

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

@jiayuasu jiayuasu changed the title [CI] Upgrade Docker CI to macos 13 [CI] Remove macos from Docker build Oct 23, 2024
@jiayuasu jiayuasu marked this pull request as ready for review October 24, 2024 01:15
@jiayuasu jiayuasu merged commit 944b9ea into master Oct 24, 2024
6 checks passed
@jiayuasu jiayuasu deleted the macos-13 branch October 24, 2024 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants