This hook was added in 278f94a8b62052ed790bb1c6832268eb79a6d03d and
72758fef22441c37f809cce098f73a21e618b9d1 to suppress spurious warnings
printed by the CLI's cli/connhelper/commandconn package;
3fb4fb83df/cli/connhelper/commandconn/commandconn.go (L203-L214)
Those logs were removed in a5ebe2282a
so we can remove the hook.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 094d1aded87d583d5d0ceac7b82955f1b445b8d1)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
full diff: https://github.com/docker/cli/compare/v24.0.2...v24.0.4
notable changes:
- ssh: fix error on commandconn close, add ping and default
- commandconn: return original error while closing
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit cc718b34443428ea15933948abf074b20aacf599)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
full diff: 9e7a6df485...36ef4d8c0d
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit ff2c8da803011c13232e64a3ab153e12539187d8)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
full diff: 4e3ac2762d...02993c407b
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit e094296f376164f3aeddda4cf744d4909560b2fb)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This should help reduce any unexpected config conflict between workers.
Signed-off-by: Justin Chadwell <me@jedevc.com>
(cherry picked from commit 6f394a06919a5271f7423e9778889841a6550832)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This means that we can run our docker-container tests in parallel again,
which can help speed up our test runs by a *significant* amount.
Signed-off-by: Justin Chadwell <me@jedevc.com>
(cherry picked from commit 601056f3a719a0fa4d437d07a7214bda9ba25e95)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit b9e25e82cfbbe7c413ddc4ebe6f88d81c02a9db3)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
#7 [internal] load metadata for docker.io/library/golang:1.20.6-buster
#7 ERROR: docker.io/library/golang:1.20.6-buster: not found
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
(cherry picked from commit 1123bfed103887ace2b2020796083323e22cbb3f)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
A temporary workaround for "http: invalid Host header" introduced in
go 1.20.6.
Signed-off-by: Justin Chadwell <me@jedevc.com>
(cherry picked from commit c4bec0546686a8978c692de97f31cf9e460e1af0)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
client: define a "dummy" hostname to use for local connections
fixes "http: invalid Host header" errors when compiling with go1.20.6
or go1.19.11
full diff: https://github.com/docker/docker/compare/v24.0.2...36e9e796c6fc
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
(cherry picked from commit 8a3a646c6130937eba92caca09a6beb4d2a1bcda)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Although a builder from the store cannot be created unless
it has a valid name, this is not the case for a Docker context.
We should skip name validation when checking a node from the
store and fall back to finding one from Docker context instead.
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
(cherry picked from commit b1c5449428436271eaa71b143e9682372ec238a4)
We need to resolve the strip the cwd:// prefix before attempting to
resolve the dockerfile. Otherwise, we'll get the cwd:// prefix in the
dockerfile name, which isn't stripped out later.
Signed-off-by: Justin Chadwell <me@jedevc.com>
(cherry picked from commit 431732f5d1f029f14adbf5828ae9798c0b9b3936)