Merge pull request #549 from flatcar/tormath1/azure

azure: remove SAS URL generation
flatcar · Aug 7, 2024 · 8be093b · 8be093b
2 parents ac3b487 + e46c085
commit 8be093b
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 16 deletions.
diff --git a/cmd/plume/prerelease.go b/cmd/plume/prerelease.go
@@ -329,15 +329,6 @@ func azurePreRelease(ctx context.Context, client *http.Client, src *storage.Buck
 		if err != nil {
 			return err
 		}
-		sas, err := azure.SignBlob(client, container, blobName)
-		if err != nil {
-			plog.Fatalf("signing failed: %v", err)
-		}
-		url := azure.BlobURL(client, container, blobName)
-		plog.Noticef("Generated SAS: %q from %q for %q", sas, url, specChannel)
-		imageInfo.Azure = &azureImageInfo{
-			ImageName: sas, // the SAS URL can be used for publishing and for testing with kola via --azure-blob-url
-		}
 	}
 	return nil
 }

diff --git a/cmd/plume/release.go b/cmd/plume/release.go
@@ -405,7 +405,6 @@ func doAzure(ctx context.Context, client *http.Client, src *storage.Bucket, spec
 
 		plog.Printf("Signing %q in %q on %v...", blobName, container, environment.CloudName)
 
-		var url string
 		blobExists, err := azure.BlobExists(client, container, blobName)
 		if err != nil {
 			plog.Fatalf("failed to check if blob %q in account %q container %q exists: %v", blobName, spec.Azure.StorageAccount, container, err)
@@ -415,12 +414,6 @@ func doAzure(ctx context.Context, client *http.Client, src *storage.Bucket, spec
 			plog.Notice("Blob does not exist, skipping.")
 			return
 		}
-		url, err = azure.SignBlob(client, container, blobName)
-		if err != nil {
-			plog.Fatalf("signing failed: %v", err)
-		}
-		plog.Noticef("Generated SAS: %q for %q", url, specChannel)
-		plog.Noticef("Please update the SKU manually (or try to automate this step)!")
 	}
 }