Skip to content

Fail certificate setup when Match fails and no renewal will run - #820

Merged
marionbarker merged 2 commits into
devfrom
fix/fail-certs-when-match-fails
Nov 12, 2026
Merged

marionbarker merged 2 commits into
devfrom
fix/fail-certs-when-match-fails

Conversation

@bjorkert

@bjorkert bjorkert commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

Create Certificates ignores the exit status of the certs lane so that an expired Distribution certificate can be nuked and recreated. When Match rejects the certificate stored in Match-Secrets while the team has another valid Distribution certificate, no renewal runs, the job succeeds and the build fails later at signing. The certs step now records whether the lane failed, and the job fails with an error saying to delete Match-Secrets when no renewal will follow and FORCE_NUKE_CERTS is not set. The certs lane checks the watch identifiers before match, so a missing watch setup fails with the existing setup message. Port of LoopKit/LoopWorkspace#573, adapted for the watch identifiers.

@marionbarker marionbarker left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approve by comparison with same file in LoopWorkspace, v3.14.12 ignoring white space differences.

This was tested prior to merging LoopWorkspace PR 573

@marionbarker
marionbarker merged commit ae04931 into dev Nov 12, 2026
2 checks passed
@marionbarker
marionbarker deleted the fix/fail-certs-when-match-fails branch November 12, 2026 16:25
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