Skip to content

Commit

Permalink
nydusify: enable pigz by default
Browse files Browse the repository at this point in the history
We should use pigz for supporting parallel gzip decompression, so that
improve the conversion speed when unpack gzip layer for source image.

We still allow users to specify the env `CONTAINERD_DISABLE_PIGZ=1` to
disable the feature when encounter any decompression error.

See https://github.com/containerd/containerd/blob/33c0eafb17dff6de43f4a530fcf05b02ed24330b/archive/compression/compression.go#L261

Signed-off-by: Yan Song <imeoer@linux.alibaba.com>
  • Loading branch information
imeoer committed Feb 28, 2023
1 parent c69a43e commit 2a83e06
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions contrib/nydusify/cmd/nydusify.go
Original file line number Diff line number Diff line change
Expand Up @@ -874,11 +874,6 @@ func main() {
},
}

// With linux/arm64 platform, containerd/compression prioritizes `unpigz`
// to decompress tar.gz file, which may generate corrupted data somehow.
// Keep the same behavior with x86_64 platform by disabling pigz.
os.Setenv("CONTAINERD_DISABLE_PIGZ", "1")

if !utils.IsSupportedArch(runtime.GOARCH) {
logrus.Fatal("Nydusify can only work under architecture 'amd64' and 'arm64'")
}
Expand Down

0 comments on commit 2a83e06

Please sign in to comment.