Skip to content

Commit

Permalink
fix(buildah): original ENTRYPOINT/CMD lost on build
Browse files Browse the repository at this point in the history
Signed-off-by: Ilya Lesikov <ilya@lesikov.com>
  • Loading branch information
ilya-lesikov committed Aug 8, 2022
1 parent 859ddb9 commit 1eebc64
Showing 1 changed file with 32 additions and 4 deletions.
36 changes: 32 additions & 4 deletions pkg/build/stage/docker_instructions.go
Expand Up @@ -2,6 +2,8 @@ package stage

import (
"context"
"encoding/json"
"fmt"
"sort"

"github.com/werf/werf/pkg/config"
Expand Down Expand Up @@ -85,17 +87,43 @@ func (s *DockerInstructionsStage) PrepareImage(ctx context.Context, c Conveyor,
imageCommitChangeOptions.AddWorkdir(s.instructions.Workdir)
imageCommitChangeOptions.AddHealthCheck(s.instructions.HealthCheck)
} else {
stageImage.Builder.StapelStageBuilder().
AddVolumes(s.instructions.Volume).
builder := stageImage.Builder.StapelStageBuilder()

builder.AddVolumes(s.instructions.Volume).
AddExpose(s.instructions.Expose).
AddEnvs(s.instructions.Env).
AddLabels(s.instructions.Label).
SetCmd([]string{s.instructions.Cmd}).
SetEntrypoint([]string{s.instructions.Entrypoint}).
SetUser(s.instructions.User).
SetWorkdir(s.instructions.Workdir).
SetHealthcheck(s.instructions.HealthCheck)

if ep, err := CmdOrEntrypointStringToSlice(s.instructions.Entrypoint); err != nil {
return fmt.Errorf("error converting ENTRYPOINT from string to slice: %w", err)
} else {
builder.SetEntrypoint(ep)
}

if cmd, err := CmdOrEntrypointStringToSlice(s.instructions.Cmd); err != nil {
return fmt.Errorf("error converting CMD from string to slice: %w", err)
} else {
builder.SetCmd(cmd)
}
}

return nil
}

func CmdOrEntrypointStringToSlice(cmdOrEntrypoint string) ([]string, error) {
var result []string
if len(cmdOrEntrypoint) > 0 {
if string(cmdOrEntrypoint[0]) == "[" && string(cmdOrEntrypoint[len(cmdOrEntrypoint)-1]) == "]" {
if err := json.Unmarshal([]byte(cmdOrEntrypoint), &result); err != nil {
return nil, fmt.Errorf("error parsing to the JSON array: %w", err)
}
} else {
result = []string{cmdOrEntrypoint}
}
}

return result, nil
}

0 comments on commit 1eebc64

Please sign in to comment.