Skip to content

vendor: github.com/moby/buildkit v0.20.0#49495

Merged
vvoland merged 1 commit into
moby:masterfrom
vvoland:update-buildkit
Feb 19, 2025
Merged

vendor: github.com/moby/buildkit v0.20.0#49495
vvoland merged 1 commit into
moby:masterfrom
vvoland:update-buildkit

Conversation

@vvoland
Copy link
Copy Markdown
Contributor

@vvoland vvoland commented Feb 19, 2025

full diff: moby/buildkit@v0.20.0-rc3...v0.20.0

Update BuildKit to v0.20.0

Signed-off-by: Paweł Gronowski [email protected]

Timeout: c.Timeout,
Client: tracing.DefaultClient,
Timeout: c.Timeout,
UserAgent: bkversion.UserAgent(),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Saw the user agent was added; does the user agent and/or should it be matching "docker engine" or "buildkit"? @jsternberg @tonistiigi

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is the same user-agent used for registry requests and ADD. The value is configurable by the importer (but unrelated to this PR).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah, thanks; wasn't sure if it's something we had do wire up to pass through our user-agent.

(but perhaps something for a future release)

Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@vvoland vvoland merged commit af898ab into moby:master Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants