Skip to content

ci: bump tflint to v0.62.1 and skip aws plugin attestation - #402

Open
l50 wants to merge 1 commit into
dreadnode:mainfrom
l50:upstream/ci-tflint-attestation-crash
Open

ci: bump tflint to v0.62.1 and skip aws plugin attestation#402
l50 wants to merge 1 commit into
dreadnode:mainfrom
l50:upstream/ci-tflint-attestation-crash

Conversation

@l50

@l50 l50 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Key Changes:

  • Upgraded TFLint from v0.61.0 to v0.62.1 to resolve a CI panic during plugin initialization
  • Disabled artifact attestation verification for the AWS ruleset plugin to prevent sigstore-go crashes
  • Documented the workarounds inline to explain the reasoning for future maintainers

Changed:

  • TFLint version pin - Bumped tflint_version to v0.62.1 in .github/workflows/pre-commit.yaml because v0.61.0 panicked (sigstore-go nil TlogEntries) when the plugin attestation fetch was forbidden in restricted-token CI, while v0.62.0+ gracefully ignores forbidden attestation-fetch errors
  • AWS plugin attestation - Added signature = "none" to the AWS ruleset plugin config in .hooks/linters/.tflint.hcl to skip attestation verification that crashes tflint --init in CI; the plugin remains pinned by version and fetched over HTTPS from the official release, and this requires tflint >= v0.62.0

@dreadnode-renovate-bot dreadnode-renovate-bot Bot added area/github Changes made to github actions area/pre-commit Changes made to pre-commit hooks labels Aug 3, 2026
**Changed:**

- Upgraded TFLint from v0.61.0 to v0.62.1 in `pre-commit.yaml` to avoid the sigstore-go nil TlogEntries panic that occurred when plugin attestation fetch was forbidden in restricted-token CI
- Consolidated pre-commit execution into a single run step, removing the separate check-only hooks stage and the associated `SKIP` environment variables now that attestation issues are resolved
- Updated autofix patch capture condition to reference only the remaining precommit step
- Set `signature = "none"` on the AWS tflint plugin in `.tflint.hcl` to skip attestation verification during `tflint --init`, while keeping the plugin pinned by version and fetched over HTTPS

**Removed:**

- Removed the separate "Run check-only hooks" step from `pre-commit.yaml`
@l50
l50 force-pushed the upstream/ci-tflint-attestation-crash branch from fb2d8d9 to fe96639 Compare August 3, 2026 20:18
@l50 l50 changed the title ci: fix tflint plugin attestation panic in restricted-token CI ci: bump tflint to v0.62.1 and skip aws plugin attestation Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/github Changes made to github actions area/pre-commit Changes made to pre-commit hooks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant