Merge pull request #2238 from k8s-infra-cherrypick-robot/cherry-pick-2237-to-release-1.27

k8s-ci-robot · web-flow · commit 735e748c29df · 2025-11-08T06:24:52.000-08:00
[release-1.27] Revert "fix: respect secretName during volume creation"
diff --git a/pkg/blob/controllerserver.go b/pkg/blob/controllerserver.go
@@ -443,7 +443,7 @@ func (d *Driver) CreateVolume(ctx context.Context, req *csi.CreateVolumeRequest)
 	}
 
 	accountOptions.Name = accountName
-	if len(secrets) == 0 && (useDataPlaneAPI || secretName != "") {
+	if len(secrets) == 0 && useDataPlaneAPI {
 		if accountKey == "" {
 			if accountName, accountKey, err = d.GetStorageAccesskey(ctx, accountOptions, secrets, secretName, secretNamespace); err != nil {
 				return nil, status.Errorf(codes.Internal, "failed to GetStorageAccesskey on account(%s) rg(%s), error: %v", accountOptions.Name, accountOptions.ResourceGroup, err)

Original file line number	Diff line number	Diff line change
`@@ -443,7 +443,7 @@ func (d Driver) CreateVolume(ctx context.Context, req csi.CreateVolumeRequest)`
`443`	`443`	`}`
`444`	`444`
`445`	`445`	`accountOptions.Name = accountName`
`446`		`- if len(secrets) == 0 && (useDataPlaneAPI \|\| secretName != "") {`
	`446`	`+ if len(secrets) == 0 && useDataPlaneAPI {`
`447`	`447`	`if accountKey == "" {`
`448`	`448`	`if accountName, accountKey, err = d.GetStorageAccesskey(ctx, accountOptions, secrets, secretName, secretNamespace); err != nil {`
`449`	`449`	`return nil, status.Errorf(codes.Internal, "failed to GetStorageAccesskey on account(%s) rg(%s), error: %v", accountOptions.Name, accountOptions.ResourceGroup, err)`