Skip to content

terraform: rename the validate job to tf-validate - #83

Merged
cpitzi merged 1 commit into
mainfrom
terraform-job-name
Aug 11, 2026
Merged

terraform: rename the validate job to tf-validate#83
cpitzi merged 1 commit into
mainfrom
terraform-job-name

Conversation

@cpitzi

@cpitzi cpitzi commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

The terraform workflow added in #82 named its validation job validate, which
is also the job id in ci.yml. A job id becomes the check-run context, so the
repo published two check runs called validate — and validate is a required
context
on main via fleet-ops/required-checks.json.

This was not a deadlock: the terraform job is gated on the changes detector and
skips on PRs that touch no terraform, and a skipped job counts as passing. But it
makes "did validate pass?" ambiguous for the ruleset, and a required context
that is not exactly the thing it appears to name is the specific failure mode
this repo's required-checks discipline exists to prevent (#27, #71).

Renames the job to tf-validate, leaving validate unambiguously ci.yml's. The
context that phase 2 will require is gate, which is unaffected and still
reports on every PR.

The gate job's references use needs['tf-validate'] rather than
needs.tf-validate: a hyphen in a context key parses as subtraction inside a
${{ }} expression.

Refs #81.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@cpitzi
cpitzi enabled auto-merge (squash) August 11, 2026 05:34
@cpitzi
cpitzi merged commit bebdb84 into main Aug 11, 2026
6 checks passed
@cpitzi
cpitzi deleted the terraform-job-name branch August 11, 2026 05:34
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