Match Docker pull/push by defaulting to progress write to stdout

Fixes: containers#21181

Signed-off-by: Daniel J Walsh <[email protected]>

cmd/podman/images/pull.go

@@ -172,7 +172,7 @@ func imagePull(cmd *cobra.Command, args []string) error {
 	pullOptions.OciDecryptConfig = decConfig
 
 	if !pullOptions.Quiet {
-		pullOptions.Writer = os.Stderr
+		pullOptions.Writer = os.Stdout
 	}
 
 	// Let's do all the remaining Yoga in the API to prevent us from

cmd/podman/images/push.go

@@ -191,7 +191,7 @@ func imagePush(cmd *cobra.Command, args []string) error {
 	}
 
 	if !pushOptions.Quiet {
-		pushOptions.Writer = os.Stderr
+		pushOptions.Writer = os.Stdout
 	}
 
 	signingCleanup, err := common.PrepareSigning(&pushOptions.ImagePushOptions,

test/e2e/pull_test.go

@@ -38,6 +38,18 @@ var _ = Describe("Podman pull", func() {
 		Expect(session.ErrorToString()).To(ContainSubstring("unauthorized: access to the requested resource is not authorized"))
 	})
 
+	It("podman pull with tag all output to stdout", func() {
+		session := podmanTest.Podman([]string{"pull", "quay.io/libpod/testdigest_v2s2:20200210"})
+		session.WaitWithDefaultTimeout()
+		Expect(session).Should(ExitCleanly())
+		Expect(session.ErrorToString()).To(BeEmpty())
+		Expect(session.OutputToString()).To(ContainSubstring("Trying to pull quay.io/libpod/testdigest_v2s2:20200210"))
+
+		session = podmanTest.Podman([]string{"rmi", "testdigest_v2s2:20200210"})
+		session.WaitWithDefaultTimeout()
+		Expect(session).Should(ExitCleanly())
+	})
+
 	It("podman pull with tag --quiet", func() {
 		session := podmanTest.Podman([]string{"pull", "-q", "quay.io/libpod/testdigest_v2s2:20200210"})
 		session.WaitWithDefaultTimeout()