Skip to content

Trust the rook formula before brew-smoke reads it - #14

Merged
samyakLambda merged 1 commit into
mainfrom
fix/brew-smoke-trust
Oct 30, 2026
Merged

samyakLambda merged 1 commit into
mainfrom
fix/brew-smoke-trust

Conversation

@samyakLambda

Copy link
Copy Markdown
Collaborator

The 0.1.17 brew-smoke (run 41035793238) failed on macos-arm before installing anything:

Refusing to load formula lambdatest/rook/rook from untrusted tap lambdatest/rook.
Run `brew trust --formula lambdatest/rook/rook` or `brew trust lambdatest/rook` to trust it.

The macos-15 image now ships Homebrew 7.0.11, which won't load a formula from a third-party tap until it is trusted. brew install lambdatest/rook/rook trusts the formula it names (the Linux log shows Trusted formula lambdatest/rook/rook), but the smoke runs brew deps first. The Linux job passed only because setup-homebrew had already trusted the tap.

This adds brew trust --formula lambdatest/rook/rook after brew tap in both jobs. Users are unaffected: the README and the install docs use the fully qualified name, and the docs already say brew install rook can be rejected as untrusted. The 0.1.17 bottles (arm64_sequoia, x86_64_linux) are published and in the formula.

After merge: gh workflow run brew-smoke.yml -f expected-version=0.1.17, since a re-run of the failed run would reuse the old workflow file.

🤖 Generated with Claude Code

Homebrew 7.0.11, preinstalled on the macos-15 image, refuses to load a
formula from a third-party tap until it is trusted, so the 0.1.17 smoke
failed at `brew deps` with "Refusing to load formula lambdatest/rook/rook
from untrusted tap" (run 41035793238). `brew install lambdatest/rook/rook`
trusts the formula it names, as it does for users, but `brew deps` runs
first. The Linux job passed only because setup-homebrew trusted the tap.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@samyakLambda
samyakLambda merged commit cbd2795 into main Oct 30, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant