Skip to content

Update Dockerfile syntax to docker/dockerfile:1#3534

Merged
kichristensen merged 1 commit intogetporter:mainfrom
kichristensen:dockerSyntax
Dec 29, 2025
Merged

Update Dockerfile syntax to docker/dockerfile:1#3534
kichristensen merged 1 commit intogetporter:mainfrom
kichristensen:dockerSyntax

Conversation

@kichristensen
Copy link
Contributor

What does this change

Updates default Docker syntax from docker/dockerfile-upstream:1.4.0 to docker/dockerfile:1.

This modernizes Porter's Dockerfile syntax to use the more current and recommended syntax version. The change affects:

  • Default syntax constant in pkg/build/dockerfile-generator.go
  • All buildkit templates
  • Test data Dockerfiles
  • Documentation examples

What issue does it fix

Closes #3533

Notes for the reviewer

All tests pass. The new syntax docker/dockerfile:1 is the recommended format and automatically uses the latest stable version.

Checklist

  • Did you write tests?
  • Did you write documentation?
  • Did you change porter.yaml or a storage document record? Update the corresponding schema file.
  • If this is your first pull request, please add your name to the bottom of our Contributors list. Thank you for making Porter better! 🙇‍♀️

@kichristensen kichristensen requested a review from a team as a code owner December 28, 2025 08:49
Replace docker/dockerfile-upstream:1.4.0 with docker/dockerfile:1
across all Dockerfiles, templates, test data, and documentation.

- Update DefaultDockerfileSyntax constant
- Update buildkit templates
- Update test data Dockerfiles
- Update documentation examples

Signed-off-by: Kim Christensen <kimworking@gmail.com>
@kichristensen kichristensen changed the title chore: Update Dockerfile syntax to docker/dockerfile:1 Update Dockerfile syntax to docker/dockerfile:1 Dec 28, 2025
@kichristensen kichristensen self-assigned this Dec 28, 2025
@kichristensen kichristensen merged commit 8c182cc into getporter:main Dec 29, 2025
47 checks passed
@kichristensen kichristensen deleted the dockerSyntax branch December 29, 2025 10:41
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.

Docker Syntax Default

2 participants