github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

Merge pull request #748 from miaoyq/del-tmpdir

Browse Source 
Delete the temporary diractory at the end of the function "ApplyLayer"
This commit is contained in:
Phil Estes 2017-04-20 00:10:35 -05:00 committed by GitHub
commit a12b20ebc4
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
rootfs/apply.go
View File

@ -4,6 +4,7 @@ import (
"context" "context"
"io" "io"
"io/ioutil" "io/ioutil"
"os"
"github.com/containerd/containerd" "github.com/containerd/containerd"
"github.com/containerd/containerd/archive" "github.com/containerd/containerd/archive"
@ -41,6 +42,7 @@ func ApplyLayer(snapshots snapshot.Snapshotter, mounter Mounter, rd io.Reader, p
if err != nil { if err != nil {
return "", errors.Wrapf(err, "creating temporary directory failed") return "", errors.Wrapf(err, "creating temporary directory failed")
} }
defer os.RemoveAll(dir)
// TODO(stevvooe): Choose this key WAY more carefully. We should be able to // TODO(stevvooe): Choose this key WAY more carefully. We should be able to
// create collisions for concurrent, conflicting unpack processes but we // create collisions for concurrent, conflicting unpack processes but we