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

Fix ctr delete to remove snapshot

Browse Source 
Fix the behavior of removing snapshot on container delete.
Adds a flag to keep the snapshot if desired.

Signed-off-by: Derek McGowan <derek@mcgstyle.net>
This commit is contained in:
Derek McGowan 2017-06-26 13:38:08 -07:00
parent 31322f9fbb
commit 7f9910d04e
No known key found for this signature in database
GPG Key ID: F58C5D0A4405ACDB
1 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
cmd/ctr/delete.go
View File

@ -11,6 +11,12 @@ var deleteCommand = cli.Command{
Name: "delete",
Usage: "delete an existing container",
ArgsUsage: "CONTAINER",
Flags: []cli.Flag{
cli.BoolFlag{
Name: "keep-snapshot",
Usage: "do not clean up snapshot with container",
},
},
Action: func(context *cli.Context) error {
ctx, cancel := appContext(context)
defer cancel()
@ -18,13 +24,17 @@ var deleteCommand = cli.Command{
if err != nil {
return err
}
deleteOpts := []containerd.DeleteOpts{}
if !context.Bool("keep-snapshot") {
deleteOpts = append(deleteOpts, containerd.WithRootFSDeletion)
}
container, err := client.LoadContainer(ctx, context.Args().First())
if err != nil {
return err
}
task, err := container.Task(ctx, nil)
if err != nil {
return container.Delete(ctx)
return container.Delete(ctx, deleteOpts...)
}
status, err := task.Status(ctx)
if err != nil {
@ -34,7 +44,7 @@ var deleteCommand = cli.Command{
if _, err := task.Delete(ctx); err != nil {
return err
}
return container.Delete(ctx)
return container.Delete(ctx, deleteOpts...)
}
return fmt.Errorf("cannot delete a container with an existing task")
},