Merge pull request #8653 from rhatdan/buildah

Honor the --layers flag

cmd/podman/images/build.go

@@ -116,6 +116,7 @@ func buildFlags(cmd *cobra.Command) {
 	// --layers flag
 	flag = layerFlags.Lookup("layers")
 	useLayersVal := useLayers()
+	buildOpts.Layers = useLayersVal == "true"
 	if err := flag.Value.Set(useLayersVal); err != nil {
 		logrus.Errorf("unable to set --layers to %v: %v", useLayersVal, err)
 	}
@@ -275,11 +276,7 @@ func buildFlagsWrapperToOptions(c *cobra.Command, contextDir string, flags *buil
 			}
 		}
 	}
-	// Check to see if the BUILDAH_LAYERS environment variable is set and
+	flags.Layers = buildOpts.Layers
-	// override command-line.
-	if _, ok := os.LookupEnv("BUILDAH_LAYERS"); ok {
-		flags.Layers = true
-	}
 
 	// `buildah bud --layers=false` acts like `docker build --squash` does.
 	// That is all of the new layers created during the build process are

pkg/api/handlers/compat/images_build.go

@@ -71,6 +71,7 @@ func BuildImage(w http.ResponseWriter, r *http.Request) {
 		ForceRm     bool     `schema:"forcerm"`
 		HTTPProxy   bool     `schema:"httpproxy"`
 		Labels      string   `schema:"labels"`
+		Layers      bool     `schema:"layers"`
 		MemSwap     int64    `schema:"memswap"`
 		Memory      int64    `schema:"memory"`
 		NetworkMode string   `schema:"networkmode"`
@@ -165,6 +166,7 @@ func BuildImage(w http.ResponseWriter, r *http.Request) {
 		Registry:                       query.Registry,
 		IgnoreUnrecognizedInstructions: true,
 		Quiet:                          query.Quiet,
+		Layers:                         query.Layers,
 		Isolation:                      buildah.IsolationChroot,
 		Compression:                    archive.Gzip,
 		Args:                           buildArgs,

pkg/bindings/images/build.go

@@ -41,6 +41,9 @@ func Build(ctx context.Context, containerFiles []string, options entities.BuildO
 	if options.NoCache {
 		params.Set("nocache", "1")
 	}
+	if options.Layers {
+		params.Set("layers", "1")
+	}
 	//	 TODO cachefrom
 	if options.PullPolicy == buildah.PullAlways {
 		params.Set("pull", "1")

test/system/070-build.bats

@@ -381,6 +381,48 @@ a${random3}z"
     run_podman rmi -f build_test
 }
 
+@test "podman build --layers test" {
+    rand_content=$(random_string 50)
+    tmpdir=$PODMAN_TMPDIR/build-test
+    run mkdir -p $tmpdir
+    containerfile=$tmpdir/Containerfile
+    cat >$containerfile <<EOF
+FROM $IMAGE
+RUN echo $rand_content
+EOF
+
+    # Build twice to make sure second time uses cache
+    run_podman build -t build_test $tmpdir
+    if [[ "$output" =~ "Using cache" ]]; then
+        is "$output" "[no instance of 'Using cache']" "no cache used"
+    fi
+
+    run_podman build -t build_test $tmpdir
+    is "$output" ".*cache" "used cache"
+
+    run_podman build -t build_test --layers=true $tmpdir
+    is "$output" ".*cache" "used cache"
+
+    run_podman build -t build_test --layers=false $tmpdir
+    if [[ "$output" =~ "Using cache" ]]; then
+        is "$output" "[no instance of 'Using cache']" "no cache used"
+    fi
+
+    BUILDAH_LAYERS=false run_podman build -t build_test $tmpdir
+    if [[ "$output" =~ "Using cache" ]]; then
+        is "$output" "[no instance of 'Using cache']" "no cache used"
+    fi
+
+    BUILDAH_LAYERS=false run_podman build -t build_test --layers=1 $tmpdir
+    is "$output" ".*cache" "used cache"
+
+    BUILDAH_LAYERS=1 run_podman build -t build_test --layers=false $tmpdir
+    if [[ "$output" =~ "Using cache" ]]; then
+        is "$output" "[no instance of 'Using cache']" "no cache used"
+    fi
+
+    run_podman rmi -a --force
+}
 
 function teardown() {
     # A timeout or other error in 'build' can leave behind stale images