Do not use beta API for hyperdisk in multi-writer mode.

karkunpavan · karkunpavan · commit 8081b51ed8dc · 2024-11-07T14:13:16.000Z
diff --git a/pkg/gce-cloud-provider/compute/gce-compute.go b/pkg/gce-cloud-provider/compute/gce-compute.go
@@ -651,7 +651,7 @@ func (cloud *CloudProvider) insertRegionalDisk(
 		gceAPIVersion = GCEAPIVersionV1
 	)
 
-	if multiWriter {
+	if multiWriter && !strings.Contains(params.DiskType, "hyperdisk") {
 		gceAPIVersion = GCEAPIVersionBeta
 	}
 
@@ -778,7 +778,7 @@ func (cloud *CloudProvider) insertZonalDisk(
 		opName        string
 		gceAPIVersion = GCEAPIVersionV1
 	)
-	if multiWriter {
+	if multiWriter && !strings.Contains(params.DiskType, "hyperdisk") {
 		gceAPIVersion = GCEAPIVersionBeta
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -651,7 +651,7 @@ func (cloud *CloudProvider) insertRegionalDisk(`
`651`	`651`	`gceAPIVersion = GCEAPIVersionV1`
`652`	`652`	`)`
`653`	`653`
`654`		`- if multiWriter {`
	`654`	`+ if multiWriter && !strings.Contains(params.DiskType, "hyperdisk") {`
`655`	`655`	`gceAPIVersion = GCEAPIVersionBeta`
`656`	`656`	`}`
`657`	`657`
`@@ -778,7 +778,7 @@ func (cloud *CloudProvider) insertZonalDisk(`
`778`	`778`	`opName string`
`779`	`779`	`gceAPIVersion = GCEAPIVersionV1`
`780`	`780`	`)`
`781`		`- if multiWriter {`
	`781`	`+ if multiWriter && !strings.Contains(params.DiskType, "hyperdisk") {`
`782`	`782`	`gceAPIVersion = GCEAPIVersionBeta`
`783`	`783`	`}`
`784`	`784`