Skip to content

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

Merged
mkultraWasHere merged 2 commits into
dreadnode:mainfrom
l50:upstream/ci-tflint-attestation-crash
Aug 13, 2026
Merged

ci: bump tflint to v0.62.1 and skip aws plugin attestation#402
mkultraWasHere merged 2 commits 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
@mkultraWasHere
mkultraWasHere added this pull request to the merge queue Aug 13, 2026
Merged via the queue into dreadnode:main with commit 3594256 Aug 13, 2026
9 checks passed
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.

2 participants