Merge branch 'main' into refactor/snapshot-embedded-containerd

Signed-off-by: Robbie Buxton <138501839+rbpdt@users.noreply.github.com>
containerd · Aug 11, 2023 · 2897250 · 2897250
2 parents 209aaf5 + 45af8ff
commit 2897250
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/snapshot/snapshot.go b/snapshot/snapshot.go
@@ -316,8 +316,9 @@ func (o *snapshotter) commit(ctx context.Context, isRemote bool, name, key strin
 		return err
 	}
 
+	rollback := true
 	defer func() {
-		if err != nil {
+		if rollback {
 			if rerr := t.Rollback(); rerr != nil {
 				log.G(ctx).WithError(rerr).Warn("failed to rollback transaction")
 			}
@@ -342,6 +343,7 @@ func (o *snapshotter) commit(ctx context.Context, isRemote bool, name, key strin
 		return fmt.Errorf("failed to commit snapshot: %w", err)
 	}
 
+	rollback = false
 	return t.Commit()
 }