Skip to content

Commit 5060ac9

Browse files
Merge pull request #10114 from barbacbd/remove-gcp-endpoints-fg
no-jira: Remove the GCP Custom Endpoints Feature Gate
2 parents 6661dac + f256cf8 commit 5060ac9

File tree

2 files changed

+1
-24
lines changed

2 files changed

+1
-24
lines changed
Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,12 @@
11
package validation
22

33
import (
4-
"k8s.io/apimachinery/pkg/util/validation/field"
5-
6-
features "github.com/openshift/api/features"
74
"github.com/openshift/installer/pkg/types"
85
"github.com/openshift/installer/pkg/types/featuregates"
96
)
107

118
// GatedFeatures determines all of the install config fields that should
129
// be validated to ensure that the proper featuregate is enabled when the field is used.
1310
func GatedFeatures(c *types.InstallConfig) []featuregates.GatedInstallConfigFeature {
14-
g := c.GCP
15-
return []featuregates.GatedInstallConfigFeature{
16-
{
17-
FeatureGateName: features.FeatureGateGCPCustomAPIEndpointsInstall,
18-
Condition: g.Endpoint != nil,
19-
Field: field.NewPath("platform", "gcp", "endpoint"),
20-
},
21-
}
11+
return []featuregates.GatedInstallConfigFeature{}
2212
}

pkg/types/validation/featuregate_test.go

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import (
1010
"github.com/openshift/installer/pkg/types"
1111
"github.com/openshift/installer/pkg/types/azure"
1212
"github.com/openshift/installer/pkg/types/dns"
13-
gcptypes "github.com/openshift/installer/pkg/types/gcp"
1413
"github.com/openshift/installer/pkg/types/vsphere"
1514
)
1615

@@ -20,18 +19,6 @@ func TestFeatureGates(t *testing.T) {
2019
installConfig *types.InstallConfig
2120
expected string
2221
}{
23-
{
24-
name: "GCP Custom API Endpoints is not allowed without Feature Gates",
25-
installConfig: func() *types.InstallConfig {
26-
c := validInstallConfig()
27-
c.GCP = validGCPPlatform()
28-
c.GCP.Endpoint = &gcptypes.PSCEndpoint{
29-
Name: "test-endpoint",
30-
}
31-
return c
32-
}(),
33-
expected: `^platform.gcp.endpoint: Forbidden: this field is protected by the GCPCustomAPIEndpointsInstall feature gate which must be enabled through either the TechPreviewNoUpgrade or CustomNoUpgrade feature set$`,
34-
},
3522
{
3623
name: "AWS UserProvisionedDNS is not allowed without Feature Gates",
3724
installConfig: func() *types.InstallConfig {

0 commit comments

Comments
 (0)