From bae0d88ae7f1a3d02346246aa260f1e7b5d75cb4 Mon Sep 17 00:00:00 2001 From: Derek McGowan Date: Thu, 4 Nov 2021 09:41:31 -0700 Subject: [PATCH] Add error logging on cleanup Check the errors for task and container deletion during restart test cleanup. Signed-off-by: Derek McGowan --- integration/client/restart_monitor_test.go | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/integration/client/restart_monitor_test.go b/integration/client/restart_monitor_test.go index ba8f5d7c1..fa400cb2c 100644 --- a/integration/client/restart_monitor_test.go +++ b/integration/client/restart_monitor_test.go @@ -156,13 +156,21 @@ version = 2 if err != nil { t.Fatal(err) } - defer container.Delete(ctx, WithSnapshotCleanup) + defer func() { + if err := container.Delete(ctx, WithSnapshotCleanup); err != nil { + t.Logf("failed to delete container: %v", err) + } + }() task, err := container.NewTask(ctx, empty()) if err != nil { t.Fatal(err) } - defer task.Delete(ctx, WithProcessKill) + defer func() { + if _, err := task.Delete(ctx, WithProcessKill); err != nil { + t.Logf("failed to delete task: %v", err) + } + }() if err := task.Start(ctx); err != nil { t.Fatal(err)