From c3d1dcc932076c15b65b8b9acfff1d47ded2ebf9 Mon Sep 17 00:00:00 2001
From: Matt Moore <mattmoor@chainguard.dev>
Date: Tue, 16 Jul 2024 18:32:55 -0700
Subject: [PATCH] Create `remote.Push` (#1978)

This exposes a function with the same signature as `remote.{Tag,Put}` using `Taggable` but that has the semantics of `remote.Write[Index` wrt writing the consituent elements of the taggable.

Signed-off-by: Matt Moore <mattmoor@chainguard.dev>
---
 pkg/v1/remote/write.go | 30 ++++++++++++++----------------
 1 file changed, 14 insertions(+), 16 deletions(-)

diff --git a/pkg/v1/remote/write.go b/pkg/v1/remote/write.go
index 30fe59051..1167cb793 100644
--- a/pkg/v1/remote/write.go
+++ b/pkg/v1/remote/write.go
@@ -45,14 +45,7 @@ type Taggable interface {
 
 // Write pushes the provided img to the specified image reference.
 func Write(ref name.Reference, img v1.Image, options ...Option) (rerr error) {
-	o, err := makeOptions(options...)
-	if err != nil {
-		return err
-	}
-	if o.progress != nil {
-		defer func() { o.progress.Close(rerr) }()
-	}
-	return newPusher(o).Push(o.context, ref, img)
+	return Push(ref, img, options...)
 }
 
 // writer writes the elements of an image to a remote image reference.
@@ -656,14 +649,7 @@ func scopesForUploadingImage(repo name.Repository, layers []v1.Layer) []string {
 // WriteIndex will attempt to push all of the referenced manifests before
 // attempting to push the ImageIndex, to retain referential integrity.
 func WriteIndex(ref name.Reference, ii v1.ImageIndex, options ...Option) (rerr error) {
-	o, err := makeOptions(options...)
-	if err != nil {
-		return err
-	}
-	if o.progress != nil {
-		defer func() { o.progress.Close(rerr) }()
-	}
-	return newPusher(o).Push(o.context, ref, ii)
+	return Push(ref, ii, options...)
 }
 
 // WriteLayer uploads the provided Layer to the specified repo.
@@ -711,3 +697,15 @@ func Put(ref name.Reference, t Taggable, options ...Option) error {
 	}
 	return newPusher(o).Put(o.context, ref, t)
 }
+
+// Push uploads the given Taggable to the specified reference.
+func Push(ref name.Reference, t Taggable, options ...Option) (rerr error) {
+	o, err := makeOptions(options...)
+	if err != nil {
+		return err
+	}
+	if o.progress != nil {
+		defer func() { o.progress.Close(rerr) }()
+	}
+	return newPusher(o).Push(o.context, ref, t)
+}