Skip to content

Commit 39ef8b2

Browse files
fix(ci): drop unused build-args passthrough so the heap ceiling can't be overridden
1 parent 7c0db52 commit 39ef8b2

1 file changed

Lines changed: 4 additions & 9 deletions

File tree

.github/actions/docker-build/action.yml

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,6 @@ inputs:
1919
tags:
2020
description: Comma-separated list of tags to push.
2121
required: true
22-
build-args:
23-
description: Newline-separated build args forwarded to the builder.
24-
required: false
25-
default: ''
2622

2723
# Registry logins must already have run in the calling job — both backends read
2824
# the same ~/.docker/config.json. provenance/sbom stay off everywhere: the
@@ -44,7 +40,6 @@ runs:
4440
platforms: ${{ inputs.platforms }}
4541
push: true
4642
tags: ${{ inputs.tags }}
47-
build-args: ${{ inputs.build-args }}
4843
provenance: false
4944
sbom: false
5045

@@ -60,7 +55,9 @@ runs:
6055
# runners have 16 GB total and `next build` was getting OOM-killed (exit
6156
# 137) partway through. Only app.Dockerfile declares the arg; the other
6257
# images ignore it harmlessly, so it is set unconditionally rather than
63-
# threaded per-image through every call site.
58+
# threaded per-image through every call site. This is deliberately the only
59+
# build arg the action passes — a general passthrough input would let a
60+
# caller re-raise the ceiling and silently restore the OOM.
6461
- name: Build and push (GitHub)
6562
if: inputs.provider != '' && inputs.provider != 'blacksmith'
6663
uses: docker/build-push-action@53b7df96c91f9c12dcc8a07bcb9ccacbed38856a # v7
@@ -70,8 +67,6 @@ runs:
7067
platforms: ${{ inputs.platforms }}
7168
push: true
7269
tags: ${{ inputs.tags }}
73-
build-args: |
74-
NEXT_BUILD_HEAP_MB=4096
75-
${{ inputs.build-args }}
70+
build-args: NEXT_BUILD_HEAP_MB=4096
7671
provenance: false
7772
sbom: false

0 commit comments

Comments
 (0)