c8d/delete: Require --force when deleting platforms

vvoland · vvoland · commit 23bfbdffc013 · 2025-05-21T19:38:43.000+02:00
When deleting specific platforms from a multi-platform image, there's
potential risk as platform-specific content might be shared across
images/containers.

For now, require `force` parameter to make the user aware.

Signed-off-by: Paweł Gronowski &lt;pawel.gronowski@docker.com&gt;
diff --git a/daemon/containerd/image_delete.go b/daemon/containerd/image_delete.go
@@ -65,6 +65,10 @@ func (i *ImageService) ImageDelete(ctx context.Context, imageRef string, options
 		}
 	}()
 
+	if len(options.Platforms) > 0 && !options.Force {
+		return nil, cerrdefs.ErrInvalidArgument.WithMessage("Deleting specific platforms is currently considered potentially unsafe because platform-specific content might be shared across images/containers. Use --force to override.")
+	}
+
 	force := options.Force
 	prune := options.PruneChildren
 

Original file line number	Diff line number	Diff line change
`@@ -65,6 +65,10 @@ func (i *ImageService) ImageDelete(ctx context.Context, imageRef string, options`
`65`	`65`	`}`
`66`	`66`	`}()`
`67`	`67`
	`68`	`+ if len(options.Platforms) > 0 && !options.Force {`
	`69`	`+ return nil, cerrdefs.ErrInvalidArgument.WithMessage("Deleting specific platforms is currently considered potentially unsafe because platform-specific content might be shared across images/containers. Use --force to override.")`
	`70`	`+ }`
	`71`	`+`
`68`	`72`	`force := options.Force`
`69`	`73`	`prune := options.PruneChildren`
`70`	`74`