Merge pull request #3905 from thaJeztah/improve_buildkit_error

cmd/docker: improve error message if BUILDKIT_ENABLED=0
This commit is contained in:
Sebastiaan van Stijn 2022-12-09 14:24:46 +01:00 committed by GitHub
commit 990674901b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 24 additions and 26 deletions

View File

@ -18,34 +18,30 @@ const (
builderDefaultPlugin = "buildx" builderDefaultPlugin = "buildx"
buildxMissingWarning = `DEPRECATED: The legacy builder is deprecated and will be removed in a future release. buildxMissingWarning = `DEPRECATED: The legacy builder is deprecated and will be removed in a future release.
Install the buildx component to build images with BuildKit: Install the buildx component to build images with BuildKit:
https://docs.docker.com/go/buildx/ https://docs.docker.com/go/buildx/`
`
buildkitDisabledWarning = `DEPRECATED: The legacy builder is deprecated and will be removed in a future release.
BuildKit is currently disabled; enabled it by removing the DOCKER_BUILDKIT=0
environment-variable.`
buildxMissingError = `ERROR: BuildKit is enabled but the buildx component is missing or broken. buildxMissingError = `ERROR: BuildKit is enabled but the buildx component is missing or broken.
Install the buildx component to build images with BuildKit: Install the buildx component to build images with BuildKit:
https://docs.docker.com/go/buildx/ https://docs.docker.com/go/buildx/`
`
) )
func newBuilderError(warn bool, err error) error { func newBuilderError(errorMsg string, pluginLoadErr error) error {
var errorMsg string if pluginmanager.IsNotFound(pluginLoadErr) {
if warn {
errorMsg = buildxMissingWarning
} else {
errorMsg = buildxMissingError
}
if pluginmanager.IsNotFound(err) {
return errors.New(errorMsg) return errors.New(errorMsg)
} }
if err != nil { if pluginLoadErr != nil {
return fmt.Errorf("%w\n\n%s", err, errorMsg) return fmt.Errorf("%w\n\n%s", pluginLoadErr, errorMsg)
} }
return fmt.Errorf("%s", errorMsg) return errors.New(errorMsg)
} }
//nolint:gocyclo //nolint:gocyclo
func processBuilder(dockerCli command.Cli, cmd *cobra.Command, args, osargs []string) ([]string, []string, []string, error) { func processBuilder(dockerCli command.Cli, cmd *cobra.Command, args, osargs []string) ([]string, []string, []string, error) {
var useLegacy, useBuilder, useAlias bool var buildKitDisabled, useBuilder, useAlias bool
var envs []string var envs []string
// check DOCKER_BUILDKIT env var is present and // check DOCKER_BUILDKIT env var is present and
@ -56,7 +52,7 @@ func processBuilder(dockerCli command.Cli, cmd *cobra.Command, args, osargs []st
return args, osargs, nil, errors.Wrap(err, "DOCKER_BUILDKIT environment variable expects boolean value") return args, osargs, nil, errors.Wrap(err, "DOCKER_BUILDKIT environment variable expects boolean value")
} }
if !enabled { if !enabled {
useLegacy = true buildKitDisabled = true
} else { } else {
useBuilder = true useBuilder = true
} }
@ -84,10 +80,10 @@ func processBuilder(dockerCli command.Cli, cmd *cobra.Command, args, osargs []st
return args, osargs, nil, nil return args, osargs, nil, nil
} }
if useLegacy { if buildKitDisabled {
// display warning if not wcow and continue // display warning if not wcow and continue
if dockerCli.ServerInfo().OSType != "windows" { if dockerCli.ServerInfo().OSType != "windows" {
_, _ = fmt.Fprintln(dockerCli.Err(), newBuilderError(true, nil)) _, _ = fmt.Fprintf(dockerCli.Err(), "%s\n\n", buildkitDisabledWarning)
} }
return args, osargs, nil, nil return args, osargs, nil, nil
} }
@ -98,12 +94,13 @@ func processBuilder(dockerCli command.Cli, cmd *cobra.Command, args, osargs []st
perr = plugin.Err perr = plugin.Err
} }
if perr != nil { if perr != nil {
// if builder enforced with DOCKER_BUILDKIT=1, cmd must fail if plugin missing or broken // if builder is enforced with DOCKER_BUILDKIT=1, cmd must fail
// if the plugin is missing or broken.
if useBuilder { if useBuilder {
return args, osargs, nil, newBuilderError(false, perr) return args, osargs, nil, newBuilderError(buildxMissingError, perr)
} }
// otherwise, display warning and continue // otherwise, display warning and continue
_, _ = fmt.Fprintln(dockerCli.Err(), newBuilderError(true, perr)) _, _ = fmt.Fprintf(dockerCli.Err(), "%s\n\n", newBuilderError(buildxMissingWarning, perr))
return args, osargs, nil, nil return args, osargs, nil, nil
} }

View File

@ -155,6 +155,7 @@ func TestBuildkitDisabled(t *testing.T) {
output.Assert(t, b.String(), map[int]func(string) error{ output.Assert(t, b.String(), map[int]func(string) error{
0: output.Suffix("DEPRECATED: The legacy builder is deprecated and will be removed in a future release."), 0: output.Suffix("DEPRECATED: The legacy builder is deprecated and will be removed in a future release."),
1: output.Suffix("BuildKit is currently disabled; enabled it by removing the DOCKER_BUILDKIT=0"),
}) })
} }

View File

@ -36,12 +36,12 @@ func TestBuildFromContextDirectoryWithTag(t *testing.T) {
withWorkingDir(dir)) withWorkingDir(dir))
defer icmd.RunCommand("docker", "image", "rm", "myimage") defer icmd.RunCommand("docker", "image", "rm", "myimage")
const buildxMissingWarning = `DEPRECATED: The legacy builder is deprecated and will be removed in a future release. const buildkitDisabledWarning = `DEPRECATED: The legacy builder is deprecated and will be removed in a future release.
Install the buildx component to build images with BuildKit: BuildKit is currently disabled; enabled it by removing the DOCKER_BUILDKIT=0
https://docs.docker.com/go/buildx/ environment-variable.
` `
result.Assert(t, icmd.Expected{Err: buildxMissingWarning}) result.Assert(t, icmd.Expected{Err: buildkitDisabledWarning})
output.Assert(t, result.Stdout(), map[int]func(string) error{ output.Assert(t, result.Stdout(), map[int]func(string) error{
0: output.Prefix("Sending build context to Docker daemon"), 0: output.Prefix("Sending build context to Docker daemon"),
1: output.Suffix("Step 1/4 : FROM registry:5000/alpine:3.6"), 1: output.Suffix("Step 1/4 : FROM registry:5000/alpine:3.6"),