Justin Chadwell

Results 161 comments of Justin Chadwell

Since https://github.com/moby/buildkit/pull/2937, syntax directives on the line after a shebang is now supported: https://github.com/moby/buildkit/blob/dc706a966d050323082c44de9c5bfe49b7251191/frontend/dockerfile/parser/directives.go#L103-L111 However, this applies **only** for syntax directives, this doesn't change the syntax of the dockerfile to...

See https://www.docker.com/blog/generate-sboms-with-buildkit/ :tada: It's released, and ready to use.

The actual error message you're seeing doesn't seem to be an HTTPS error: ``` ailed to solve: our-registrybase-images/debian:latest: failed to do request: Head "https://our-registry/v2/base-images/debian/manifests/latest": dial tcp 192.168.1.1:443: connect: no route...

> The problem is its trying to use HTTPS, even tho its in insecure registries. Sure, that's possible there could be a bug here, but that's not what the error...

cc @vvoland who did some work on this recently as well: - https://github.com/moby/buildkit/pull/4293 - https://github.com/moby/buildkit/pull/4299

Looks like this was originally added in https://github.com/moby/buildkit/pull/615/files#diff-d77a9a4e3d44aebce1412917bb17e4cdf95b7cfcb8bebca87b68aea48eded183R31 I remember this being significant, see: https://github.com/moby/buildkit/blob/d6e142600ee58a6ec45f946b25af28f9a70c6dbc/client/ociindex/ociindex.go#L128-L132 And maybe also https://github.com/docker/buildx/pull/1456. This is the field used to do lookups in this file...

@tompizmor no activity recently on this. If anyone's interested, we'd be happy to take a PR to do this. Also, x-linking this to https://github.com/moby/buildkit/pull/4269 (this is related to upgrading to...

What's the command you're using to build the image? Can you reproduce, or was this a one-off failure?

Spent some time looking into this, it's specifically related to the codepath here: https://github.com/moby/buildkit/blob/2713491176c5f9fa64dd26ea7efa2f20557db576/solver/jobs.go#L808-L812 This happens when `CacheMap.PreprocessFunc` is set (which could explain why we maybe only seem to see...

Yeah, that tracks with my observations as well - the issue definitely seems related to edge merging / shared vertices, which happens when multiple build requests are run at the...