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

UnixPB: Only Install CMake on mac where not already present. #3544

Merged
merged 15 commits into from
May 3, 2024

Conversation

steelhead31
Copy link
Contributor

@steelhead31 steelhead31 commented May 2, 2024

This should fix the failing GHA, as the MacOS 14 runner, already has cmake installed.

Fixes #3549

Checklist

@steelhead31
Copy link
Contributor Author

Raised upstream bug for the MacOS14 runner: actions/runner-images#9777

@steelhead31 steelhead31 marked this pull request as ready for review May 2, 2024 11:40
Copy link
Member

@sxa sxa left a comment

Choose a reason for hiding this comment

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

I think the location is different for x64 and aarch64, although given that we're only seeing the problem on one of them (AFAIK) I'm ok with this for now.

I'm curious though - do we know why this causes the problem? I'd have hoped that state: present would make it do the right thing and only install if it's not there - almost feels like cmake has been installed on the runners in a manner which isn't compatible with brew.
We should also consider that if it's an "unofficial" install we've no idea where it came from or what version it is.

@steelhead31
Copy link
Contributor Author

I think the location is different for x64 and aarch64, although given that we're only seeing the problem on one of them (AFAIK) I'm ok with this for now.

I'm curious though - do we know why this causes the problem? I'd have hoped that state: present would make it do the right thing and only install if it's not there - almost feels like cmake has been installed on the runners in a manner which isn't compatible with brew. We should also consider that if it's an "unofficial" install we've no idea where it came from or what version it is.

@sxa Im not sure how the github runners are constructed, Im assuming without ansible, but I can ask the question upstream, on my bug... an alternate solution for the github action is to remove it from the runner, so the playbook can install the correct version, as this only occurs on the github runner, it works as expected everywhere else.

@steelhead31 steelhead31 marked this pull request as draft May 2, 2024 14:53
@steelhead31
Copy link
Contributor Author

Going to quickly test the removal from the runner, and allowing the playbook to run normally :)

@steelhead31 steelhead31 marked this pull request as ready for review May 2, 2024 16:01
@steelhead31
Copy link
Contributor Author

Removing cmake from the runner is not straightforward, so the ansible check is currently a tidier solution, I'll ask some questions upstream.

@karianna
Copy link
Contributor

karianna commented May 3, 2024

Ran the Mac OS X 11 action twice but I can't see the clear reason for failure.

…s/MacOSX.yml

Co-authored-by: Martijn Verburg <martijnverburg@gmail.com>
@steelhead31 steelhead31 requested a review from karianna May 3, 2024 07:47
@steelhead31
Copy link
Contributor Author

Ran the Mac OS X 11 action twice but I can't see the clear reason for failure.

@karianna it appears to be running , but the performance is pretty poor compared to the 14 runner. Macos 11 was end of life, in November 2023, so if this action keeps being problematic, it might be worth considering replacing macos 11 with 13, or dropping it entirely.

@steelhead31
Copy link
Contributor Author

Also a quick note that the macos-11 runner will be deprecated from 28th June see https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners

The macos-11 label has been deprecated and will no longer be available after 6/28/2024.

@gdams
Copy link
Member

gdams commented May 3, 2024

Also a quick note that the macos-11 runner will be deprecated from 28th June see https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners

The macos-11 label has been deprecated and will no longer be available after 6/28/2024.

I'd just kill it, maybe switch it out with macos-13?

Replace macos11 with 13, and improve cmake check logic
@sxa
Copy link
Member

sxa commented May 3, 2024

Also a quick note that the macos-11 runner will be deprecated from 28th June see https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners

The macos-11 label has been deprecated and will no longer be available after 6/28/2024.

I'd just kill it, maybe switch it out with macos-13?

Yeah if it's being deprecated that seems like a reasonable course of action

@gdams gdams dismissed karianna’s stale review May 3, 2024 16:01

fixed now

@gdams gdams merged commit 0b0553e into adoptium:master May 3, 2024
12 checks passed
@steelhead31 steelhead31 deleted the fix_macos_gha branch May 7, 2024 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GitHub Actions For Checking Ansible Playbooks
4 participants