testutil: fix call order in already-exists check

ProviderFromRef call is already making the HEAD request
to upstream registry to get digest. Avoid doing that if
image already exists in mirror anyway.

Signed-off-by: Tonis Tiigi <[email protected]>

Author: tonistiigi

util/testutil/integration/run.go

@@ -236,6 +236,11 @@ func copyImagesLocal(t *testing.T, host string, images map[string]string) error
 		}
 		localImageCache[host][to] = struct{}{}
 
+		// already exists check
+		if _, _, err := docker.NewResolver(docker.ResolverOptions{}).Resolve(context.TODO(), host+"/"+to); err == nil {
+			continue
+		}
+
 		var desc ocispecs.Descriptor
 		var provider content.Provider
 		var err error
@@ -255,12 +260,6 @@ func copyImagesLocal(t *testing.T, host string, images map[string]string) error
 			}
 		}
 
-		// already exists check
-		_, _, err = docker.NewResolver(docker.ResolverOptions{}).Resolve(context.TODO(), host+"/"+to)
-		if err == nil {
-			continue
-		}
-
 		ingester, err := contentutil.IngesterFromRef(host + "/" + to)
 		if err != nil {
 			return err