Merge pull request #1299 from vrothberg/revert-ba3a9c0f770b

Revert "libimage: pull: do not enforce pull if local image matches"

libimage/pull.go

@@ -497,26 +497,16 @@ func (r *Runtime) copySingleImageFromRegistry(ctx context.Context, imageName str
 
 	customPlatform := len(options.Architecture)+len(options.OS)+len(options.Variant) > 0
 	if customPlatform && pullPolicy != config.PullPolicyAlways && pullPolicy != config.PullPolicyNever {
-		// Unless the specified platform matches the local image, we
-		// must pessimistically assume that the local image has an
-		// invalid architecture (see containers/podman/issues/10682).
-		// Hence, whenever the user requests a custom platform, set the
-		// pull policy to "newer" to make sure we're pulling down the
+		// Unless the pull policy is always/never, we must
+		// pessimistically assume that the local image has an invalid
+		// architecture (see containers/podman/issues/10682).  Hence,
+		// whenever the user requests a custom platform, set the pull
+		// policy to "newer" to make sure we're pulling down the
 		// correct image.
 		//
 		// NOTE that this is will even override --pull={false,never}.
-		localImageMatches := false
-		if localImage != nil {
-			_, matches, err := localImage.matchesPlatform(ctx, options.OS, options.Architecture, options.Variant)
-			if err != nil {
-				return nil, err
-			}
-			localImageMatches = matches
-		}
-		if !localImageMatches {
-			pullPolicy = config.PullPolicyNewer
-			logrus.Debugf("Enforcing pull policy to %q to pull custom platform (arch: %q, os: %q, variant: %q) - local image may mistakenly specify wrong platform", pullPolicy, options.Architecture, options.OS, options.Variant)
-		}
+		pullPolicy = config.PullPolicyNewer
+		logrus.Debugf("Enforcing pull policy to %q to pull custom platform (arch: %q, os: %q, variant: %q) - local image may mistakenly specify wrong platform", pullPolicy, options.Architecture, options.OS, options.Variant)
 	}
 
 	if pullPolicy == config.PullPolicyNever {

libimage/pull_test.go

@@ -5,11 +5,9 @@ import (
 	"fmt"
 	"os"
 	goruntime "runtime"
-	"strings"
 	"testing"
 
 	"github.com/containers/common/pkg/config"
-	"github.com/sirupsen/logrus"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 )
@@ -96,27 +94,6 @@ func TestPullPlatforms(t *testing.T) {
 	require.NoError(t, err, "pull busybox")
 	require.Len(t, pulledImages, 1)
 
-	// Now re-pull with the platform explicitly set in the pull options. It
-	// should not repull an image (or perform a "newer" check) though but
-	// resolve to the local image.
-	//
-	// See containers/podman/issues/17063.
-	func() { // Anonymous function to make sure logrus is reset even on failure.
-		builder := strings.Builder{}
-		logrus.SetOutput(&builder)
-		logrus.SetLevel(logrus.DebugLevel)
-		defer builder.Reset()
-		defer logrus.SetOutput(os.Stderr)
-		defer logrus.SetLevel(logrus.InfoLevel)
-
-		pullOptions.Architecture = localArch
-		pullOptions.OS = localOS
-		pulledImages, err := runtime.Pull(ctx, withTag, config.PullPolicyMissing, pullOptions)
-		require.NoError(t, err, "pull busybox with same platform as before")
-		require.Len(t, pulledImages, 1)
-		require.NotContains(t, builder.String(), "local image may mistakenly specify wrong platform")
-	}()
-
 	// Repulling with a bogus architecture should yield an error and not
 	// choose the local image.
 	pullOptions.Architecture = "bogus"