vendor: containerd v1.6.2, buildkit v0.10.0

[thaJeztah]

vendor: google.golang.org/grpc v1.44.0

full diff: grpc/grpc-go@v1.38.0...v1.44.0

vendor: github.com/containerd/containerd v1.6.2

full diff: containerd/containerd@v1.5.10...v1.6.2

vendor: github.com/prometheus/procfs v0.7.3

un-pinning the dependency to let go modules resolve the version to use.

full diff: prometheus/procfs@v0.0.11...v0.7.3

vendor: github.com/prometheus/client_golang v1.11.0

un-pin the dependency to let go modules resolve the version

full diff: prometheus/client_golang@v1.6.0...v1.11.0

vendor: github.com/moby/buildkit v0.10.0

full diff: moby/buildkit@9f254e1...v0.10.0

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

[codecov-commenter]

Codecov Report

Merging #3485 (5f74020) into master (0e3c9b2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3485   +/-   ##
=======================================
  Coverage   59.01%   59.01%           
=======================================
  Files         284      284           
  Lines       23833    23833           
=======================================
  Hits        14066    14066           
  Misses       8908     8908           
  Partials      859      859           

[thaJeztah]

@rumpl @crazy-max PTAL

[crazy-max]

I wonder if the buildkit dependency is still something required here. It looks like we depend on it for only two small pkgs that could be directly integrated here:

cli/cmd/docker/docker.go

Line 18 in 6745f62

"github.com/moby/buildkit/util/appcontext"

cli/cmd/docker/docker.go

Lines 198 to 201 in 6745f62

	go func() {
	// override SIGTERM handler so we let the plugin shut down first
	<-appcontext.Context().Done()
	}()

---

cli/cli/command/image/build/dockerignore.go

Line 8 in a32cd16

"github.com/moby/buildkit/frontend/dockerfile/dockerignore"

cli/cli/command/image/build/dockerignore.go

Line 25 in a32cd16

return dockerignore.ReadAll(f)

The dockerignore one needs more thoughts though (split parser/frontend from buildkit): moby/buildkit#2734 (comment)

[thaJeztah]

Yes, I was also looking at some bits for the dockerignore, and for context I want to look at getting rid of pkg/urlutil, which looks like it made its way into other areas where it shouldn't be used (and I think there's equivalents to that in buildkit); https://github.com/moby/moby/tree/babb0c14fd1281b309de46c3481a5bea88b8031d/pkg/urlutil

[thaJeztah]

@crazy-max those things ok to do in a follow-up (if possible?)? Looks like we may not be able to resolve them immediately (in this repository), or at least only partial

[crazy-max]

Yes let's think about that in a follow-up

[thaJeztah]

Thx!