Skip to content

github builder architecture#24604

Merged
crazy-max merged 1 commit intodocker:mainfrom
crazy-max:github-builder-arch
Apr 2, 2026
Merged

github builder architecture#24604
crazy-max merged 1 commit intodocker:mainfrom
crazy-max:github-builder-arch

Conversation

@crazy-max
Copy link
Copy Markdown
Member

Description

Architecture page for the GitHub Builder.

Reviews

  • Technical review
  • Editorial review
  • Product review

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 1, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 5e05df4
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69ce3948edb6750008dba9ba
😎 Deploy Preview https://deploy-preview-24604--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@crazy-max crazy-max force-pushed the github-builder-arch branch from c38a5ef to e90ec90 Compare April 2, 2026 08:06
@crazy-max crazy-max marked this pull request as ready for review April 2, 2026 08:12
@crazy-max crazy-max requested a review from dvdksn as a code owner April 2, 2026 08:12
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great 🚀 let's just use the already-defined version param for consistency

@crazy-max crazy-max force-pushed the github-builder-arch branch from e90ec90 to a5b6741 Compare April 2, 2026 08:17
@crazy-max crazy-max requested a review from dvdksn April 2, 2026 08:25
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM oops just noticed one more thing


## Examples

## Dockerfile-oriented image build
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn Apr 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just noticed some heading levels were off by one!

Suggested change
## Dockerfile-oriented image build
### Dockerfile-oriented image build

platform fan-out, metadata generation, provenance, SBOM generation, signing,
and final manifest creation.

## Bake-oriented local output
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
## Bake-oriented local output
### Bake-oriented local output

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah good catch!

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max crazy-max force-pushed the github-builder-arch branch from a5b6741 to 5e05df4 Compare April 2, 2026 09:39
@crazy-max crazy-max requested a review from dvdksn April 2, 2026 09:39
@crazy-max crazy-max merged commit 8adfb60 into docker:main Apr 2, 2026
17 of 18 checks passed
@crazy-max crazy-max deleted the github-builder-arch branch April 2, 2026 10:17
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.

2 participants