Skip to content

ci: fix post-release homebrew formula and docs update jobs - #1841

Merged
lokesh-vadlamudi merged 1 commit into
mainfrom
lvadlamudi/fix-release-automation
Jun 23, 2026
Merged

ci: fix post-release homebrew formula and docs update jobs#1841
lokesh-vadlamudi merged 1 commit into
mainfrom
lvadlamudi/fix-release-automation

Conversation

@lokesh-vadlamudi

Copy link
Copy Markdown
Collaborator

Summary

The two automation jobs that fire after a Release completes both broke on the v2.14.1 release. Neither is a code/test failure — they're release-time infra breakages, fixed independently here.

Update Homebrew Formula (update-homebrew-formula.yml)

Failed at the Update CLI Version step:

Refusing to load formula lacework/lacework-cli/lacework-cli from untrusted tap lacework/lacework-cli.
make: *** [Makefile:12: update-cli-version] Error 1

Recent Homebrew refuses to load a formula from a third-party tap unless it's explicitly trusted. The failure surfaces inside the homebrew-tap script's lint()brew audit --formula lacework-cli --strict. Since the tap is created locally in this workflow, the fix belongs here:

  • Add brew trust lacework/lacework-cli before make update-cli-version.
  • Bump Homebrew/actions/setup-homebrew@master@main (master is deprecated and becomes a hard error in Homebrew 5.2.0).

Update CLI Docs (update-cli-docs.yml)

Docs generated and committed fine; only the push died:

git@github.com: Permission denied (publickey).
fatal: Could not read from remote repository.

The step forced origin to the SSH URL (git@github.com:lacework/docs_lw.git), but CI only has the HTTPS token from actions/checkout — no SSH key is loaded. Removed the set-url override so the push reuses the token-authenticated HTTPS remote.

Notes

  • No change needed in lacework/homebrew-tap; the brew trust step covers the audit call without coupling the tap script to CI-only setup.
  • The Node 20 deprecation annotations are warnings only (jobs force-run on Node 24) and are left untouched.

- update-homebrew-formula: bump setup-homebrew@master to @main (master is
  deprecated and errors in Homebrew 5.2.0) and trust the locally-created
  lacework/lacework-cli tap so brew audit will load its formula
- update-cli-docs: stop forcing origin to the SSH remote, which failed the
  docs push with 'Permission denied (publickey)'; the token-authenticated
  HTTPS remote from actions/checkout is used instead
@lokesh-vadlamudi lokesh-vadlamudi self-assigned this Jun 23, 2026
@lokesh-vadlamudi
lokesh-vadlamudi marked this pull request as ready for review June 23, 2026 21:16
@lokesh-vadlamudi
lokesh-vadlamudi requested a review from a team as a code owner June 23, 2026 21:16
@lokesh-vadlamudi
lokesh-vadlamudi enabled auto-merge (squash) June 23, 2026 21:16
@lokesh-vadlamudi
lokesh-vadlamudi requested a review from wsli-1 June 23, 2026 21:17
@lokesh-vadlamudi
lokesh-vadlamudi merged commit fac8eac into main Jun 23, 2026
19 of 20 checks passed
@lokesh-vadlamudi
lokesh-vadlamudi deleted the lvadlamudi/fix-release-automation branch June 23, 2026 22:09
@lacework-releng lacework-releng mentioned this pull request Jul 16, 2026
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.

2 participants