Repository navigation
chore(events-processor): drop Dockerfile.staging - #822
Merged
Merged
Conversation
#811 hardened the production events-processor/Dockerfile onto the Wolfi/apko bases, which is the only reason Dockerfile.staging existed — staging needed a Wolfi build while prod was still on Debian. With both on Wolfi the two files were near-duplicates, and the production one is strictly better: immutable base pins, a sha-pinned LAGO_EXPRESSION_REF instead of a repointable tag, a single-commit fetch instead of `git clone --tags`, go mod verify, explicit CGO_ENABLED=1, and OCI labels. The staging build keeps its distinct role via build-args in lago-deploy#3611: the bases are overridden to `:latest` so staging still rides the daily lago-packages CVE rebuild, while production stays reproducible on a pinned base. Requires lago-deploy#3611 to merge first — it repoints the staging workflow at events-processor/Dockerfile. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
koikonom
approved these changes
Oct 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes
events-processor/Dockerfile.staging(47 lines, one file).Requires getlago/lago-deploy#3611 to merge first — that PR repoints the staging build workflow at
events-processor/Dockerfile. Merging this one first leaves the staging workflow pointing at a file that no longer exists.Why it can go
Dockerfile.stagingexisted for exactly one reason: staging needed a Wolfi build while production was still on Debian. #811 hardened the production Dockerfile onto the same lago-packages bases, making the two near-duplicates — and the production one is strictly better:Dockerfile(prod)Dockerfile.staging(removed):latestLAGO_EXPRESSION_REFgit checkout v0.2.0— a repointable taggit fetch --depth 1 <sha>git clone --tags(every branch and tag)go mod verify, explicitCGO_ENABLED=1Staging keeps its role
The one genuine difference — staging riding the daily lago-packages CVE rebuild while prod stays reproducible — is preserved in lago-deploy#3611 as build-args on the same Dockerfile:
One override instead of 47 duplicated lines. Staging remains the canary for base bumps, and it also gains the sha-pinned expression ref.
Already validated in production
The production Dockerfile this consolidates onto is live in both prod clusters as of today:
It cleared 13 critical + 58 high findings.
🤖 Generated with Claude Code