pkg/podman/podman, cmd/rmi: Change return values of IsToolboxImage()

The original version of the IsToolboxImage() function in
pkg/podman/podman.go returned an error when an image was not
compatible with Toolbx.

The new version returns 'false' without error on a non-Toolbx
container, so it can be distinguished when the image inspection
actually fails.

This new behavior is used in detecting such images when creating
or running Toolbx containers: f06a819

Author: DaliborKr

src/cmd/rmi.go

@@ -90,9 +90,12 @@ func rmi(cmd *cobra.Command, args []string) error {
 		}
 
 		for _, image := range args {
-			if _, err := podman.IsToolboxImage(image); err != nil {
+			if isToolboxImage, err := podman.IsToolboxImage(image); err != nil {
 				fmt.Fprintf(os.Stderr, "Error: %s\n", err)
 				continue
+			} else if !isToolboxImage {
+				fmt.Fprintf(os.Stderr, "Error: %s is not a Toolbx image\n", image)
+				continue
 			}
 
 			if err := podman.RemoveImage(image, rmiFlags.forceDelete); err != nil {

src/pkg/podman/podman.go

@@ -352,12 +352,12 @@ func IsToolboxImage(image string) (bool, error) {
 	}
 
 	if info["Labels"] == nil {
-		return false, fmt.Errorf("%s is not a Toolbx image", image)
+		return false, nil
 	}
 
 	labels := info["Labels"].(map[string]interface{})
 	if labels["com.github.containers.toolbox"] != "true" && labels["com.github.debarshiray.toolbox"] != "true" {
-		return false, fmt.Errorf("%s is not a Toolbx image", image)
+		return false, nil
 	}
 
 	return true, nil