Merge pull request #1673 from amacaskill/fix-storage-pools-metric

amacaskill · web-flow · commit d0825e5f7494 · 2024-04-18T14:51:23.000-07:00
Remove short variable declaration from validateStoragePools
diff --git a/pkg/gce-pd-csi-driver/controller.go b/pkg/gce-pd-csi-driver/controller.go
@@ -288,7 +288,8 @@ func (gceCS *GCEControllerServer) CreateVolume(ctx context.Context, req *csi.Cre
 		gceAPIVersion = gce.GCEAPIVersionBeta
 	}
 
-	if err := validateStoragePools(req, params, gceCS.CloudProvider.GetDefaultProject()); err != nil {
+	err = validateStoragePools(req, params, gceCS.CloudProvider.GetDefaultProject())
+	if err != nil {
 		return nil, status.Errorf(codes.InvalidArgument, "CreateVolume failed to validate storage pools: %v", err)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -288,7 +288,8 @@ func (gceCS GCEControllerServer) CreateVolume(ctx context.Context, req csi.Cre`
`288`	`288`	`gceAPIVersion = gce.GCEAPIVersionBeta`
`289`	`289`	`}`
`290`	`290`
`291`		`- if err := validateStoragePools(req, params, gceCS.CloudProvider.GetDefaultProject()); err != nil {`
	`291`	`+ err = validateStoragePools(req, params, gceCS.CloudProvider.GetDefaultProject())`
	`292`	`+ if err != nil {`
`292`	`293`	`return nil, status.Errorf(codes.InvalidArgument, "CreateVolume failed to validate storage pools: %v", err)`
`293`	`294`	`}`
`294`	`295`