feat(server): add build log

Signed-off-by: Yaroslav Pershin <62902094+iapershin@users.noreply.github.com>

Author: iapershin

server/pkg/docker/build.go

@@ -8,7 +8,6 @@ import (
 	"fmt"
 	"os/exec"
 	"path"
-	"regexp"
 
 	"github.com/djherbis/buffer"
 	"github.com/djherbis/nio/v3"
@@ -82,19 +81,20 @@ func BuildReleaseArtifacts(ctx context.Context, opts BuildReleaseArtifactsOpts,
 		}
 	}()
 
-	logboek.Context(ctx).Default().LogF("Building docker image with artifacts\n")
-	logger.Debug("Building docker image with artifacts")
+	logboek.Context(ctx).Default().LogLn("Building docker image with artifacts")
+	logger.Info("Building docker image with artifacts")
 
 	args, err := setCliArgs(serviceDockerfilePathInContext, secrets)
 	if err != nil {
 		return fmt.Errorf("unable to set cli args: %w", err), nil
 	}
 
-	if err := RunCliBuild(ctx, contextReader, opts.TarWriter, args...); err != nil {
+	if err := RunCliBuild(ctx, logger, contextReader, opts.TarWriter, args...); err != nil {
 		return fmt.Errorf("can't build artifacts: %w", err), nil
 	}
 
-	logboek.Context(ctx).Default().LogF("Build is successful\n")
+	logboek.Context(ctx).Default().LogLn("Build is successful")
+	logger.Info("Build is successful")
 
 	cleanupFunc := func() error {
 		cli, err := client.NewClientWithOpts(client.FromEnv, client.WithAPIVersionNegotiation())
@@ -107,21 +107,25 @@ func BuildReleaseArtifacts(ctx context.Context, opts BuildReleaseArtifactsOpts,
 	return nil, cleanupFunc
 }
 
-func RunCliBuild(ctx context.Context, contextReader *nio.PipeReader, tarWriter *nio.PipeWriter, args ...string) error {
+func RunCliBuild(ctx context.Context, logger hclog.Logger, contextReader *nio.PipeReader, tarWriter *nio.PipeWriter, args ...string) error {
 	finalArgs := append([]string{"buildx", "build"}, args...)
 	cmd := exec.CommandContext(ctx, "docker", finalArgs...)
 	var stdErrBuf bytes.Buffer
 	cmd.Stdout = tarWriter
 	cmd.Stdin = contextReader
 	cmd.Stderr = &stdErrBuf
 
-	if err := cmd.Run(); err != nil {
-		if len := stdErrBuf.Len(); len > 0 {
-			errSection, parseErr := extractRelevantLogs(&stdErrBuf)
-			if parseErr == nil {
-				return fmt.Errorf("build failed: %s %w", errSection.String(), err)
+	defer func() {
+		if stdErrBuf.Len() > 0 {
+			scanner := bufio.NewScanner(&stdErrBuf)
+			for scanner.Scan() {
+				logger.Info(scanner.Text())
+				logboek.Context(ctx).Default().LogLn(scanner.Text())
 			}
 		}
+	}()
+
+	if err := cmd.Run(); err != nil {
 		return fmt.Errorf("build failed: %w", err)
 	}
 
@@ -132,39 +136,6 @@ func RunCliBuild(ctx context.Context, contextReader *nio.PipeReader, tarWriter *
 	return nil
 }
 
-// this needs for parsing buildx logs due to the fact that buildx writes all to stderr
-// it will look for the first error section of the logs that contains the error
-// or if not found just lines starts with "error:" or "ERROR:"
-func extractRelevantLogs(stderr *bytes.Buffer) (bytes.Buffer, error) {
-	scanner := bufio.NewScanner(stderr)
-
-	var errSection bytes.Buffer
-	var foundSection bool
-	reSectionStart := regexp.MustCompile(`^------$`)
-	reError := regexp.MustCompile(`(?i)^error:`)
-
-	for scanner.Scan() {
-		line := scanner.Text()
-
-		if reSectionStart.MatchString(line) {
-			if foundSection {
-				break
-			}
-			foundSection = true
-		}
-
-		if foundSection || reError.MatchString(line) {
-			errSection.WriteString(line + "\n")
-		}
-	}
-
-	if err := scanner.Err(); err != nil {
-		return errSection, fmt.Errorf("error reading stderr: %w", err)
-	}
-
-	return errSection, nil
-}
-
 func setCliArgs(serviceDockerfilePathInContext string, secrets []secrets.Secret) ([]string, error) {
 	args := []string{
 		"--file", serviceDockerfilePathInContext,