Build step is always required for tagging images

This commit is contained in:
Ettore Di Giacinto
2020-12-07 19:39:56 +01:00
parent ebd18ae22c
commit 9a34296be0

View File

@@ -346,12 +346,6 @@ func (cs *LuetCompiler) buildPackageImage(image, buildertaggedImage, packageImag
return builderOpts, runnerOpts, errors.Wrap(err, "Could not generate image definition") return builderOpts, runnerOpts, errors.Wrap(err, "Could not generate image definition")
} }
noBuildInstructions := len(p.BuildSteps()) == 0 && len(p.GetPreBuildSteps()) == 0
if len(p.BuildSteps()) == 0 || noBuildInstructions {
packageImage = buildertaggedImage
}
if len(p.GetPreBuildSteps()) == 0 { if len(p.GetPreBuildSteps()) == 0 {
buildertaggedImage = image buildertaggedImage = image
} }
@@ -410,11 +404,11 @@ func (cs *LuetCompiler) buildPackageImage(image, buildertaggedImage, packageImag
} }
} }
if len(p.BuildSteps()) != 0 || len(p.BuildSteps()) == 0 && len(p.GetPreBuildSteps()) == 0 { // Even if we might not have any steps to build, we do that so we can tag the image used in this moment and use that to cache it in a registry, or in the system.
Info(pkgTag, ":whale: Generating 'package' image from", buildertaggedImage, "as", packageImage, "with build steps") // acting as a docker tag.
if err := buildAndPush(runnerOpts); err != nil { Info(pkgTag, ":whale: Generating 'package' image from", buildertaggedImage, "as", packageImage, "with build steps")
return builderOpts, runnerOpts, errors.Wrap(err, "Could not push image: "+image+" "+builderOpts.DockerFileName) if err := buildAndPush(runnerOpts); err != nil {
} return builderOpts, runnerOpts, errors.Wrap(err, "Could not push image: "+image+" "+builderOpts.DockerFileName)
} }
return builderOpts, runnerOpts, nil return builderOpts, runnerOpts, nil