New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: buildkit section #15958
build: buildkit section #15958
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
ff24533
to
a192e97
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This addition, and especially the LLB description, is so damn good I am gonna print it, frame it, and put it on a wall. I have some rephrasing suggestions, that's all.
build/extend/index.md
Outdated
A frontend is a component that takes a human-readable | ||
build format and converts it to LLB so BuildKit can execute it. Frontends can | ||
be distributed as images, and the user can target a specific version of a | ||
frontend that is guaranteed to work for the features used by their definition. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for later: I think we will want to lift the description of "what's a BuildKit frontend" to a more prominent/visible place. I was confused about why it was called frontends until I read these paragraphs, and now it totally makes sense. Let's leave it here for now though!
a192e97
to
768f6a1
Compare
768f6a1
to
0edef9f
Compare
88d096c
to
9834c19
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM once the corresponding buildkit PR is merged.
9834c19
to
6978afb
Compare
Oh more links need to be changed on docker/cli repo. Edit: PRs opened: |
6978afb
to
5df5a8a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; bunch of nits
5df5a8a
to
6bb65e9
Compare
Thanks for the review @dvdksn and @jedevc. Will wait for docker/cli#3832 to be merged to fix the htmlproofer issue. |
52f37b6
to
3d470b0
Compare
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
3d470b0
to
2bac3f3
Compare
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
needs moby/buildkit#3220needs docker/cli#3832
Adds "BuildKit" section describing the BuildKit project the most basic way with cross linking to existing pages. Also adds a "Custom Dockerfile syntax" page reworked from Dockerfile reference documentation.
Following these changes I also merged the "build enhancements", "enabling buildkit" and "creating a dockerfile" pages in build section with proper redirects and cross linking.
Signed-off-by: CrazyMax crazy-max@users.noreply.github.com