Skip to content

chore: pin cloudflare provider below 5.20.0 + lock-file check#72

Merged
danieldietzler merged 2 commits into
mainfrom
chore/pin-cloudflare-provider
Jun 16, 2026
Merged

chore: pin cloudflare provider below 5.20.0 + lock-file check#72
danieldietzler merged 2 commits into
mainfrom
chore/pin-cloudflare-provider

Conversation

@zackpollard

@zackpollard zackpollard commented Jun 16, 2026

Copy link
Copy Markdown
Member

No description provided.

5.20.0 has a bug, so constrain all worker modules to >= 5, < 5.20.0.
cloudflare-metrics and version had no committed .terraform.lock.hcl, so
their provider version was unpinned and resolved to the newest release on
each init. Add lock files (cloudflare pinned to 5.19.1) for them and refresh
the constraint metadata in the existing hello/github-approval-check locks.
Comment thread .github/workflows/test.yml Fixed
@immich-push-o-matic

immich-push-o-matic Bot commented Jun 16, 2026

Copy link
Copy Markdown

Preview Deployments (78461eb)

Worker Preview URL
github-approval-check https://github-approval-check.pr-72.dev.immich.app
hello https://hello.pr-72.dev.immich.cloud
version https://version.pr-72.dev.immich.cloud

Add deployment/scripts/check-lock-files.sh (and a tf:check-locks mise task)
that fails when a module declaring required_providers has no committed
.terraform.lock.hcl, and run it in the Test workflow.

Add a least-privilege permissions block to the Test workflow (contents: read,
plus pull-requests: read for the paths-filter job) so the new job doesn't trip
zizmor's excessive-permissions audit.
@zackpollard zackpollard force-pushed the chore/pin-cloudflare-provider branch from 123cfb2 to 78461eb Compare June 16, 2026 12:24
@danieldietzler danieldietzler merged commit 55dee4d into main Jun 16, 2026
11 checks passed
@danieldietzler danieldietzler deleted the chore/pin-cloudflare-provider branch June 16, 2026 13:07
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.

3 participants