Skip to content

Conversation

@juliankuners
Copy link
Contributor

@juliankuners juliankuners commented Sep 1, 2025

The release workflow in CI started to break, see here due to brew and cmake.

This workflow uses a MacOS image provided by this GitHub action. While the version of cmake stayed the same, I assume that the image has switched from a non-brew cmake to a brew-provided cmake, causing a conflict when building K with brew.

This pull request fixes this issue by uninstalling cmake beforehand.

@juliankuners
Copy link
Contributor Author

The fix was tested successfully in this CI run.

Copy link
Member

@RaoulSchaffranek RaoulSchaffranek left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@Robertorosmaninho Robertorosmaninho left a comment

Choose a reason for hiding this comment

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

LGTM

@juliankuners juliankuners changed the title Uninstall pre-installed cmake prior to building with brew Uninstall pre-installed cmake prior to building with brew in CI Sep 2, 2025
@juliankuners juliankuners merged commit 5e6300b into develop Sep 2, 2025
46 of 53 checks passed
@juliankuners juliankuners deleted the fix-macos-release branch September 2, 2025 11:51
automergerpr-permission-manager bot pushed a commit that referenced this pull request Sep 11, 2025
This PR allows bypassing the `macos-test` CI job to publish new K
releases.

The release workflow started failing in the `macos-build` job as of this
PR:
#4850
The failure seems unrelated to the changes introduced by the PR and
looks more like a flakiness issue in the CI due to non-deterministic
dependency resolution.

A follow-up PR fixed the `macos-build` job:
#4851

Unfortunately, the workflow is now failing in the `macos-test` job.
#4852 is an attempt to
debug and fix the tests.

The inability to publish new K-releases is currently blocking important
updates on Simbolik.
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.

4 participants