Skip to content

Commit a64c11f

Browse files
authored
Followup (#6)
* Changes remove the API version concept from GetDisk and InsertDisk, now just defaults to using the Beta API. * Cleaning up un-used variables.
1 parent a062e89 commit a64c11f

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

pkg/gce-pd-csi-driver/controller.go

-8
Original file line numberDiff line numberDiff line change
@@ -2432,10 +2432,6 @@ func createRegionalDisk(ctx context.Context, cloudProvider gce.GCECompute, name
24322432
return nil, fmt.Errorf("failed to insert regional disk: %w", err)
24332433
}
24342434

2435-
gceAPIVersion := gce.GCEAPIVersionV1
2436-
if multiWriter {
2437-
gceAPIVersion = gce.GCEAPIVersionBeta
2438-
}
24392435
// failed to GetDisk, however the Disk may already be created, the error code should be non-Final
24402436
disk, err := cloudProvider.GetDisk(ctx, project, meta.RegionalKey(name, region))
24412437
if err != nil {
@@ -2455,10 +2451,6 @@ func createSingleZoneDisk(ctx context.Context, cloudProvider gce.GCECompute, nam
24552451
return nil, fmt.Errorf("failed to insert zonal disk: %w", err)
24562452
}
24572453

2458-
gceAPIVersion := gce.GCEAPIVersionV1
2459-
if multiWriter {
2460-
gceAPIVersion = gce.GCEAPIVersionBeta
2461-
}
24622454
// failed to GetDisk, however the Disk may already be created, the error code should be non-Final
24632455
disk, err := cloudProvider.GetDisk(ctx, project, meta.ZonalKey(name, diskZone))
24642456
if err != nil {

0 commit comments

Comments
 (0)