Remove the --push option from the collect command (#67)

The option to make the `collect` command push the snapshot to remote
storage isn't used, and make our API more confusing. Let's remove it,
at least for now.

snapshots/go/cmd/snapshots/collect.go

@@ -27,7 +27,6 @@ type collectConfig struct {
 	bazelCacheGRPCInsecure bool
 	bazelStderr            bool
 	outPath                string
-	push                   bool
 	noPrint                bool
 }
 
@@ -82,28 +81,10 @@ func runCollect(args []string) error {
 	log.Println("out path:        ", cc.outPath)
 
 	// run the command
-	ctx := context.Background()
-	manifest, err := collect(cc)
-	if err != nil {
+	if _, err := collect(cc); err != nil {
 		return fmt.Errorf("failed to collect: %w", err)
 	}
 
-	if cc.push {
-		pc := getPushConfig(c)
-		pc.snapshot = manifest
-
-		obj, err := push(ctx, pc)
-		if err != nil {
-			return fmt.Errorf("failed to push snapshot: %w", err)
-		}
-
-		contentLenght, isOk := obj.GetContentLength()
-		if !isOk {
-			log.Printf("failed to get contentLenght of pushed snapshot: %s", obj.Path)
-		}
-		log.Printf("pushed snapshot of %d bytes: %s", contentLenght, obj.Path)
-	}
-
 	return nil
 }