chore: drop redundant STORAGE_DRIVER/BUILDAH_ISOLATION env vars in build-push.yaml #41

Open
opened 2026-05-01 05:58:17 +00:00 by rcsheets · 0 comments
Owner

The job-level env block in .forgejo/workflows/build-push.yaml sets:

STORAGE_DRIVER: vfs
BUILDAH_ISOLATION: chroot

Both were added as belt-and-suspenders before the storage.conf-based approach landed (#38, fix/buildah-vfs-storage-conf). Now that /etc/containers/storage.conf pins driver = "vfs" authoritatively for both buildah and podman, these env vars are redundant — the conf file is what actually does the coercion (the env var alone wasn't enough; podman reads prior driver metadata from the storage path before consulting the env, which is what triggered the original overlay-on-overlayfs failure).

Drop the env vars from the jobs.build-push.env block once we've seen ~3-4 successful build-push runs against main to confirm nothing else regresses.

The job-level `env` block in `.forgejo/workflows/build-push.yaml` sets: ```yaml STORAGE_DRIVER: vfs BUILDAH_ISOLATION: chroot ``` Both were added as belt-and-suspenders before the storage.conf-based approach landed (#38, `fix/buildah-vfs-storage-conf`). Now that `/etc/containers/storage.conf` pins `driver = "vfs"` authoritatively for both buildah and podman, these env vars are redundant — the conf file is what actually does the coercion (the env var alone wasn't enough; podman reads prior driver metadata from the storage path before consulting the env, which is what triggered the original overlay-on-overlayfs failure). Drop the env vars from the `jobs.build-push.env` block once we've seen ~3-4 successful build-push runs against `main` to confirm nothing else regresses.
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
brooktrails/pr-reviewer#41
No description provided.