Skip to content

fix: add dockerfile syntax directive for heredocs - #818

Open
Namanlashkari wants to merge 2 commits into
Kong:masterfrom
Namanlashkari:docs/dockerfile-syntax-directive
Open

fix: add dockerfile syntax directive for heredocs#818
Namanlashkari wants to merge 2 commits into
Kong:masterfrom
Namanlashkari:docs/dockerfile-syntax-directive

Conversation

@Namanlashkari

@Namanlashkari Namanlashkari commented Aug 26, 2026

Copy link
Copy Markdown

Problem

assets/Dockerfile uses RUN <<-EOF for the LuaRocks install. Docker daemons that still default to the dockerfile.v0 frontend parse the heredoc body as instructions and fail:

unknown instruction: BUILD_VERSION=$(

#730

Fix

Add # syntax=docker/dockerfile:1 as the first line so BuildKit pulls the modern frontend regardless of the daemon built-in.

Testing

Manual:

  • Confirmed assets/Dockerfile starts with # syntax=docker/dockerfile:1
  • Heredoc RUN <<-EOF block is unchanged

CI:

  • Existing Pongo image-build CI should parse the Dockerfile on current BuildKit

The luarocks install step uses RUN <<-EOF. Without
# syntax=docker/dockerfile:1, older Docker frontends parse the
heredoc body as instructions and fail.

Fix Kong#730
@Namanlashkari Namanlashkari changed the title fix(docker): add dockerfile syntax directive for heredocs fix: add dockerfile syntax directive for heredocs Aug 26, 2026
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.

1 participant