From 63a37d2e5315017ffcaf8079a523a6e4a53a1eb9 Mon Sep 17 00:00:00 2001 From: Sharif Elgamal Date: Tue, 2 Nov 2021 15:57:09 -0700 Subject: [PATCH] stop setting ACLs on preload tarball --- hack/preload-images/upload.go | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/hack/preload-images/upload.go b/hack/preload-images/upload.go index 89bbd8004b..0a0e41ab50 100644 --- a/hack/preload-images/upload.go +++ b/hack/preload-images/upload.go @@ -34,19 +34,5 @@ func uploadTarball(tarballFilename string) error { if output, err := cmd.CombinedOutput(); err != nil { return errors.Wrapf(err, "uploading %s to GCS bucket: %v\n%s", hostPath, err, string(output)) } - // Make tarball public to all users - gcsPath := fmt.Sprintf("%s/%s", gcsDest, tarballFilename) - cmd = exec.Command("gsutil", "acl", "ch", "-u", "AllUsers:R", gcsPath) - fmt.Printf("Running: %v\n", cmd.Args) - if output, err := cmd.CombinedOutput(); err != nil { - fmt.Printf(`Failed to update ACLs on this tarball in GCS. Please run - -gsutil acl ch -u AllUsers:R %s - -manually to make this link public, or rerun this script to rebuild and reupload the tarball. - - `, gcsPath) - return errors.Wrapf(err, "uploading %s to GCS bucket: %v\n%s", hostPath, err, string(output)) - } return nil }