Skip to content

Commit e593707

Browse files
Merge pull request #9944 from sadasu/update-gcp-custom-dns-featuregate
CORS-4162: Remove GCPClusterHostedDNSInstall featuregate
2 parents bcf3757 + 6aff305 commit e593707

File tree

2 files changed

+0
-16
lines changed

2 files changed

+0
-16
lines changed

pkg/types/gcp/validation/featuregates.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import (
55

66
features "github.com/openshift/api/features"
77
"github.com/openshift/installer/pkg/types"
8-
"github.com/openshift/installer/pkg/types/dns"
98
"github.com/openshift/installer/pkg/types/featuregates"
109
)
1110

@@ -14,11 +13,6 @@ import (
1413
func GatedFeatures(c *types.InstallConfig) []featuregates.GatedInstallConfigFeature {
1514
g := c.GCP
1615
return []featuregates.GatedInstallConfigFeature{
17-
{
18-
FeatureGateName: features.FeatureGateGCPClusterHostedDNSInstall,
19-
Condition: g.UserProvisionedDNS == dns.UserProvisionedDNSEnabled,
20-
Field: field.NewPath("platform", "gcp", "userProvisionedDNS"),
21-
},
2216
{
2317
FeatureGateName: features.FeatureGateGCPCustomAPIEndpointsInstall,
2418
Condition: len(g.ServiceEndpoints) > 0,

pkg/types/validation/featuregate_test.go

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -19,16 +19,6 @@ func TestFeatureGates(t *testing.T) {
1919
installConfig *types.InstallConfig
2020
expected string
2121
}{
22-
{
23-
name: "GCP UserProvisionedDNS is not allowed without Feature Gates",
24-
installConfig: func() *types.InstallConfig {
25-
c := validInstallConfig()
26-
c.GCP = validGCPPlatform()
27-
c.GCP.UserProvisionedDNS = dns.UserProvisionedDNSEnabled
28-
return c
29-
}(),
30-
expected: `^platform.gcp.userProvisionedDNS: Forbidden: this field is protected by the GCPClusterHostedDNSInstall feature gate which must be enabled through either the TechPreviewNoUpgrade or CustomNoUpgrade feature set$`,
31-
},
3222
{
3323
name: "GCP Custom API Endpoints is not allowed without Feature Gates",
3424
installConfig: func() *types.InstallConfig {

0 commit comments

Comments
 (0)