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

fix(ci): race-condition in signing macos binaries #433

Merged
merged 3 commits into from
Dec 7, 2022

Conversation

apricote
Copy link
Member

@apricote apricote commented Dec 7, 2022

By default the keychain that is used to sign macOS binaries is only unlocked for 300s. This caused the release process to fail, as building all binaries took longer than the 300s, and subsequent signing failed.

The new timeout is 3600s (1h), which should be enough to build anything we want. At the moment building all binaries takes ~8m, so we have a buffer of 52m or 650%.

Further changes made here:

  • fix(ci): show logs of gon hooks

    This makes debugging any issues encountered in these steps easier.

  • fix(ci): use similar goreleaser settings to tagged release builds

    In release builds we use --parallelism=2 and the latest goreleaser version. We should use the same settings in our dev builds.

By default the keychain that is used to sign macOS binaries is only
unlocked for 300s. This caused the release process to fail, as building
all binaries took longer than the 300s, and subsequent signing failed.

The new timeout is 3600s (1h), which should be enough to build anything we
want. At the moment building all binaries takes ~8m, so we have a buffer of
52m or 650%.
This makes debugging any issues encountered in these steps easier.
In release builds we use `--parallelism=2` and the latest goreleaser version.

We should use the same settings in our dev builds.
@apricote apricote self-assigned this Dec 7, 2022
@apricote
Copy link
Member Author

apricote commented Dec 7, 2022

@apricote apricote requested a review from LKaemmerling December 7, 2022 13:11
@apricote apricote merged commit a44a40c into main Dec 7, 2022
@apricote apricote deleted the fix-macos-signing-2 branch December 7, 2022 13:29
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