Removes -z flag from decompress since tar -x detects it automatically.

Omitting the -z flag during extraction (-x) is considered best practice as it works for both compress and non-compressed tars.

Author: mdzhigarov

fileutil/tarball_compressor.go

@@ -64,7 +64,7 @@ func (c tarballCompressor) DecompressFileToDir(tarballPath string, dir string, o
 	if err != nil {
 		return bosherr.WrapError(err, "Resolving tarball path")
 	}
-	args := []string{sameOwnerOption, "-xzf", resolvedTarballPath, "-C", dir}
+	args := []string{sameOwnerOption, "-xf", resolvedTarballPath, "-C", dir}
 	if options.StripComponents != 0 {
 		args = append(args, fmt.Sprintf("--strip-components=%d", options.StripComponents))
 	}

fileutil/tarball_compressor_test.go

@@ -206,7 +206,7 @@ var _ = Describe("tarballCompressor", func() {
 			Expect(cmdRunner.RunCommands[0]).To(Equal(
 				[]string{
 					"tar", "--no-same-owner",
-					"-xzf", tarballPath,
+					"-xf", tarballPath,
 					"-C", dstDir,
 				},
 			))
@@ -228,7 +228,7 @@ var _ = Describe("tarballCompressor", func() {
 			Expect(cmdRunner.RunCommands[0]).To(Equal(
 				[]string{
 					"tar", "--same-owner",
-					"-xzf", tarballPath,
+					"-xf", tarballPath,
 					"-C", dstDir,
 				},
 			))
@@ -246,7 +246,7 @@ var _ = Describe("tarballCompressor", func() {
 			Expect(cmdRunner.RunCommands[0]).To(Equal(
 				[]string{
 					"tar", "--no-same-owner",
-					"-xzf", tarballPath,
+					"-xf", tarballPath,
 					"-C", dstDir,
 					"some/path/in/archive",
 				},
@@ -265,7 +265,7 @@ var _ = Describe("tarballCompressor", func() {
 			Expect(cmdRunner.RunCommands[0]).To(Equal(
 				[]string{
 					"tar", "--no-same-owner",
-					"-xzf", tarballPath,
+					"-xf", tarballPath,
 					"-C", dstDir,
 					"--strip-components=3",
 				},