[Gateway API 1.2.0] Upgrade Gateway API to 1.2.0-rc2

examples/extension-server/go.mod

@@ -10,7 +10,7 @@ require (
 	google.golang.org/protobuf v1.34.2
 	k8s.io/apimachinery v0.31.1
 	sigs.k8s.io/controller-runtime v0.19.0
-	sigs.k8s.io/gateway-api v1.1.0
+	sigs.k8s.io/gateway-api v1.2.0-rc1
 )
 
 require (

examples/extension-server/go.sum

@@ -133,8 +133,8 @@ k8s.io/utils v0.0.0-20240711033017-18e509b52bc8 h1:pUdcCO1Lk/tbT5ztQWOBi5HBgbBP1
 k8s.io/utils v0.0.0-20240711033017-18e509b52bc8/go.mod h1:OLgZIPagt7ERELqWJFomSt595RzquPNLL48iOWgYOg0=
 sigs.k8s.io/controller-runtime v0.19.0 h1:nWVM7aq+Il2ABxwiCizrVDSlmDcshi9llbaFbC0ji/Q=
 sigs.k8s.io/controller-runtime v0.19.0/go.mod h1:iRmWllt8IlaLjvTTDLhRBXIEtkCK6hwVBJJsYS9Ajf4=
-sigs.k8s.io/gateway-api v1.1.0 h1:DsLDXCi6jR+Xz8/xd0Z1PYl2Pn0TyaFMOPPZIj4inDM=
-sigs.k8s.io/gateway-api v1.1.0/go.mod h1:ZH4lHrL2sDi0FHZ9jjneb8kKnGzFWyrTya35sWUTrRs=
+sigs.k8s.io/gateway-api v1.2.0-rc1 h1:ZcqCEype0Mrt3ax46t2QB+VSm8ic4/WUQkbcc6xtAOI=
+sigs.k8s.io/gateway-api v1.2.0-rc1/go.mod h1:EpNfEXNjiYfUJypf0eZ0P5iXA9ekSGWaS1WgPaM42X0=
 sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd h1:EDPBXCAspyGV4jQlpZSudPeMmr1bNJefnuqLsRAsHZo=
 sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd/go.mod h1:B8JuhiUyNFVKdsE8h686QcCxMaH6HrOAZj4vswFpcB0=
 sigs.k8s.io/structured-merge-diff/v4 v4.4.1 h1:150L+0vs/8DA78h1u02ooW1/fFq/Lwr+sGiqlzvrtq4=

go.mod

@@ -56,7 +56,7 @@ require (
 	k8s.io/kubectl v0.31.1
 	k8s.io/utils v0.0.0-20240711033017-18e509b52bc8
 	sigs.k8s.io/controller-runtime v0.19.0
-	sigs.k8s.io/gateway-api v1.1.0
+	sigs.k8s.io/gateway-api v1.2.0-rc1
 	sigs.k8s.io/mcs-api v0.1.0
 	sigs.k8s.io/yaml v1.4.0
 )

go.sum

@@ -1163,8 +1163,8 @@ sigs.k8s.io/controller-runtime v0.6.1/go.mod h1:XRYBPdbf5XJu9kpS84VJiZ7h/u1hF3gE
 sigs.k8s.io/controller-runtime v0.19.0 h1:nWVM7aq+Il2ABxwiCizrVDSlmDcshi9llbaFbC0ji/Q=
 sigs.k8s.io/controller-runtime v0.19.0/go.mod h1:iRmWllt8IlaLjvTTDLhRBXIEtkCK6hwVBJJsYS9Ajf4=
 sigs.k8s.io/controller-tools v0.3.0/go.mod h1:enhtKGfxZD1GFEoMgP8Fdbu+uKQ/cq1/WGJhdVChfvI=
-sigs.k8s.io/gateway-api v1.1.0 h1:DsLDXCi6jR+Xz8/xd0Z1PYl2Pn0TyaFMOPPZIj4inDM=
-sigs.k8s.io/gateway-api v1.1.0/go.mod h1:ZH4lHrL2sDi0FHZ9jjneb8kKnGzFWyrTya35sWUTrRs=
+sigs.k8s.io/gateway-api v1.2.0-rc1 h1:ZcqCEype0Mrt3ax46t2QB+VSm8ic4/WUQkbcc6xtAOI=
+sigs.k8s.io/gateway-api v1.2.0-rc1/go.mod h1:EpNfEXNjiYfUJypf0eZ0P5iXA9ekSGWaS1WgPaM42X0=
 sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd h1:EDPBXCAspyGV4jQlpZSudPeMmr1bNJefnuqLsRAsHZo=
 sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd/go.mod h1:B8JuhiUyNFVKdsE8h686QcCxMaH6HrOAZj4vswFpcB0=
 sigs.k8s.io/kind v0.8.1/go.mod h1:oNKTxUVPYkV9lWzY6CVMNluVq8cBsyq+UgPJdvA3uu4=

internal/gatewayapi/conformance/suite.go

@@ -15,6 +15,7 @@ import (
 // SkipTests is a list of tests that are skipped in the conformance suite.
 var SkipTests = []suite.ConformanceTest{
 	tests.GatewayStaticAddresses,
+	tests.GatewayInfrastructure,
 }
 
 func skipTestsShortNames(skipTests []suite.ConformanceTest) []string {
@@ -27,9 +28,26 @@ func skipTestsShortNames(skipTests []suite.ConformanceTest) []string {
 
 // EnvoyGatewaySuite is the conformance suite configuration for the Gateway API.
 var EnvoyGatewaySuite = suite.ConformanceOptions{
-	SupportedFeatures: features.AllFeatures,
-	ExemptFeatures: sets.New[features.SupportedFeature]().
-		Insert(features.MeshCoreFeatures.UnsortedList()...).
-		Insert(features.MeshExtendedFeatures.UnsortedList()...),
-	SkipTests: skipTestsShortNames(SkipTests),
+	SupportedFeatures: allFeatures(),
+	ExemptFeatures:    meshFeatures(),
+	SkipTests:         skipTestsShortNames(SkipTests),
+}
+
+func allFeatures() sets.Set[features.FeatureName] {
+	allFeatures := sets.New[features.FeatureName]()
+	for _, feature := range features.AllFeatures.UnsortedList() {
+		allFeatures.Insert(feature.Name)
+	}
+	return allFeatures
+}
+
+func meshFeatures() sets.Set[features.FeatureName] {
+	meshFeatures := sets.New[features.FeatureName]()
+	for _, feature := range features.MeshCoreFeatures.UnsortedList() {
+		meshFeatures.Insert(feature.Name)
+	}
+	for _, feature := range features.MeshExtendedFeatures.UnsortedList() {
+		meshFeatures.Insert(feature.Name)
+	}
+	return meshFeatures
 }

internal/gatewayapi/conformance/support_level.go

@@ -26,10 +26,18 @@ const (
 )
 
 // ExtendedFeatures is a list of supported Gateway-API features that are considered Extended.
-var ExtendedFeatures = sets.New[features.SupportedFeature]().
-	Insert(features.GatewayExtendedFeatures.UnsortedList()...).
-	Insert(features.HTTPRouteExtendedFeatures.UnsortedList()...).
-	Insert(features.MeshExtendedFeatures.UnsortedList()...)
+var ExtendedFeatures = sets.New[features.FeatureName]()
+
+func init() {
+	featureLists := sets.New[features.Feature]().
+		Insert(features.GatewayExtendedFeatures.UnsortedList()...).
+		Insert(features.HTTPRouteExtendedFeatures.UnsortedList()...).
+		Insert(features.MeshExtendedFeatures.UnsortedList()...)
+
+	for _, feature := range featureLists.UnsortedList() {
+		ExtendedFeatures.Insert(feature.Name)
+	}
+}
 
 // GetTestSupportLevel returns the SupportLevel for a conformance test.
 // The support level is determined by the highest support level of the features.
@@ -44,7 +52,7 @@ func GetTestSupportLevel(test suite.ConformanceTest) SupportLevel {
 }
 
 // GetFeatureSupportLevel returns the SupportLevel for a feature.
-func GetFeatureSupportLevel(feature features.SupportedFeature) SupportLevel {
+func GetFeatureSupportLevel(feature features.FeatureName) SupportLevel {
 	supportLevel := Core
 
 	if ExtendedFeatures.Has(feature) {

internal/gatewayapi/helpers.go

@@ -82,6 +82,7 @@ var (
 	PathMatchTypeDerefOr       = ptr.Deref[gwapiv1.PathMatchType]
 	GRPCMethodMatchTypeDerefOr = ptr.Deref[gwapiv1.GRPCMethodMatchType]
 	HeaderMatchTypeDerefOr     = ptr.Deref[gwapiv1.HeaderMatchType]
+	GRPCHeaderMatchTypeDerefOr = ptr.Deref[gwapiv1.GRPCHeaderMatchType]
 	QueryParamMatchTypeDerefOr = ptr.Deref[gwapiv1.QueryParamMatchType]
 )
 

internal/gatewayapi/route.go

@@ -608,13 +608,13 @@
 		}
 
 		for _, headerMatch := range match.Headers {
-			switch HeaderMatchTypeDerefOr(headerMatch.Type, gwapiv1.HeaderMatchExact) {
-			case gwapiv1.HeaderMatchExact:
+			switch GRPCHeaderMatchTypeDerefOr(headerMatch.Type, gwapiv1.GRPCHeaderMatchExact) {
+			case gwapiv1.GRPCHeaderMatchExact:
 				irRoute.HeaderMatches = append(irRoute.HeaderMatches, &ir.StringMatch{
 					Name:  string(headerMatch.Name),
 					Exact: ptr.To(headerMatch.Value),
 				})
-			case gwapiv1.HeaderMatchRegularExpression:
+			case gwapiv1.GRPCHeaderMatchRegularExpression:
 				if err := regex.Validate(headerMatch.Value); err != nil {
 					return nil, err
 				}

internal/gatewayapi/status/gatewayclass.go

@@ -82,12 +82,19 @@ func getSupportedFeatures(gatewaySuite suite.ConformanceOptions, skippedTests []
 
 	ret := sets.New[gwapiv1.SupportedFeature]()
 	for _, feature := range supportedFeatures.UnsortedList() {
-		ret.Insert(gwapiv1.SupportedFeature(feature))
+		ret.Insert(gwapiv1.SupportedFeature{
+			Name: gwapiv1.FeatureName(feature),
+		})
 	}
-	return sets.List(ret)
+
+	var featureList []gwapiv1.SupportedFeature
+	for feature := range ret {
+		featureList = append(featureList, feature)
+	}
+	return featureList
 }
 
-func getUnsupportedFeatures(gatewaySuite suite.ConformanceOptions, skippedTests []suite.ConformanceTest) []features.SupportedFeature {
+func getUnsupportedFeatures(gatewaySuite suite.ConformanceOptions, skippedTests []suite.ConformanceTest) []features.FeatureName {
 	unsupportedFeatures := gatewaySuite.ExemptFeatures.UnsortedList()
 
 	for _, skippedTest := range skippedTests {

internal/gatewayapi/status/gatewayclass_test.go

@@ -80,63 +80,76 @@ func TestGetSupportedFeatures(t *testing.T) {
 		{
 			name: "No exempt features",
 			gatewaySuite: suite.ConformanceOptions{
-				SupportedFeatures: sets.New[features.SupportedFeature]("Gateway", "HTTPRoute"),
-				ExemptFeatures:    sets.New[features.SupportedFeature](),
+				SupportedFeatures: sets.New[features.FeatureName]("Gateway", "HTTPRoute"),
+				ExemptFeatures:    sets.New[features.FeatureName](),
+			},
+			expectedResult: []gwapiv1.SupportedFeature{
+				{Name: "Gateway"},
+				{Name: "HTTPRoute"},
 			},
-			expectedResult: []gwapiv1.SupportedFeature{"Gateway", "HTTPRoute"},
 		},
 		{
 			name: "All features exempt",
 			gatewaySuite: suite.ConformanceOptions{
-				SupportedFeatures: sets.New[features.SupportedFeature]("Gateway", "HTTPRoute"),
-				ExemptFeatures:    sets.New[features.SupportedFeature]("Gateway", "HTTPRoute"),
+				SupportedFeatures: sets.New[features.FeatureName]("Gateway", "HTTPRoute"),
+				ExemptFeatures:    sets.New[features.FeatureName]("Gateway", "HTTPRoute"),
 			},
 			expectedResult: []gwapiv1.SupportedFeature{},
 		},
 		{
 			name: "Some features exempt",
 			gatewaySuite: suite.ConformanceOptions{
-				SupportedFeatures: sets.New[features.SupportedFeature]("Gateway", "HTTPRoute", "GRPCRoute"),
-				ExemptFeatures:    sets.New[features.SupportedFeature]("GRPCRoute"),
+				SupportedFeatures: sets.New[features.FeatureName]("Gateway", "HTTPRoute", "GRPCRoute"),
+				ExemptFeatures:    sets.New[features.FeatureName]("GRPCRoute"),
+			},
+			expectedResult: []gwapiv1.SupportedFeature{
+				{Name: "Gateway"},
+				{Name: "HTTPRoute"},
 			},
-			expectedResult: []gwapiv1.SupportedFeature{"Gateway", "HTTPRoute"},
 		},
 		{
 			name: "Some features exempt with skipped tests",
 			gatewaySuite: suite.ConformanceOptions{
-				SupportedFeatures: sets.New[features.SupportedFeature]("Gateway", "HTTPRoute", "GRPCRoute"),
-				ExemptFeatures:    sets.New[features.SupportedFeature]("GRPCRoute"),
+				SupportedFeatures: sets.New[features.FeatureName]("Gateway", "HTTPRoute", "GRPCRoute"),
+				ExemptFeatures:    sets.New[features.FeatureName]("GRPCRoute"),
 			},
 			skippedTests: []suite.ConformanceTest{
 				{
-					Features: []features.SupportedFeature{"HTTPRoute"},
+					Features: []features.FeatureName{"HTTPRoute"},
 				},
 			},
-			expectedResult: []gwapiv1.SupportedFeature{"Gateway"},
+			expectedResult: []gwapiv1.SupportedFeature{
+				{Name: "Gateway"},
+			},
 		},
 		{
 			name: "Core features remain supported with skipped extended tests",
 			gatewaySuite: suite.ConformanceOptions{
-				SupportedFeatures: sets.New[features.SupportedFeature]("Gateway", "HTTPRoute", "GatewayHTTPListenerIsolation"),
+				SupportedFeatures: sets.New[features.FeatureName]("Gateway", "HTTPRoute", "GatewayHTTPListenerIsolation"),
 			},
 			skippedTests: []suite.ConformanceTest{
 				{
-					Features: []features.SupportedFeature{"Gateway", "GatewayHTTPListenerIsolation", "HTTPRoute"},
+					Features: []features.FeatureName{"Gateway", "GatewayHTTPListenerIsolation", "HTTPRoute"},
 				},
 			},
-			expectedResult: []gwapiv1.SupportedFeature{"Gateway", "HTTPRoute"},
+			expectedResult: []gwapiv1.SupportedFeature{
+				{Name: "Gateway"},
+				{Name: "HTTPRoute"},
+			},
 		},
 		{
 			name: "Core feature removed when skipping core test",
 			gatewaySuite: suite.ConformanceOptions{
-				SupportedFeatures: sets.New[features.SupportedFeature]("Gateway", "HTTPRoute"),
+				SupportedFeatures: sets.New[features.FeatureName]("Gateway", "HTTPRoute"),
 			},
 			skippedTests: []suite.ConformanceTest{
 				{
-					Features: []features.SupportedFeature{"HTTPRoute"},
+					Features: []features.FeatureName{"HTTPRoute"},
 				},
 			},
-			expectedResult: []gwapiv1.SupportedFeature{"Gateway"},
+			expectedResult: []gwapiv1.SupportedFeature{
+				{Name: "Gateway"},
+			},
 		},
 	}
 

test/e2e/e2e_test.go

@@ -46,7 +46,7 @@ func TestE2E(t *testing.T) {
 		RunTest:              *flags.RunTest,
 		// SupportedFeatures cannot be empty, so we set it to SupportGateway
 		// All e2e tests should leave Features empty.
-		SupportedFeatures: sets.New[features.SupportedFeature](features.SupportGateway),
+		SupportedFeatures: sets.New[features.FeatureName](features.SupportGateway),
 		SkipTests: []string{
 			tests.GatewayInfraResourceTest.ShortName, // https://github.com/envoyproxy/gateway/issues/3191
 		},

test/e2e/merge_gateways/merge_gateways_test.go

@@ -47,7 +47,7 @@ func TestMergeGateways(t *testing.T) {
 		RunTest:              *flags.RunTest,
 		// SupportedFeatures cannot be empty, so we set it to SupportGateway
 		// All e2e tests should leave Features empty.
-		SupportedFeatures: sets.New[features.SupportedFeature](features.SupportGateway),
+		SupportedFeatures: sets.New[features.FeatureName](features.SupportGateway),
 		SkipTests:         []string{},
 	})
 	if err != nil {

test/e2e/multiple_gc/multiple_gc_test.go

@@ -27,7 +27,6 @@ import (
 
 func TestMultipleGC(t *testing.T) {
 	flag.Parse()
-
 	c, cfg := kubetest.NewClient(t)
 
 	if flags.RunTest != nil && *flags.RunTest != "" {
@@ -50,7 +49,7 @@ func TestMultipleGC(t *testing.T) {
 			RunTest:              *flags.RunTest,
 			// SupportedFeatures cannot be empty, so we set it to SupportGateway
 			// All e2e tests should leave Features empty.
-			SupportedFeatures: sets.New[features.SupportedFeature](features.SupportGateway),
+			SupportedFeatures: sets.New[features.FeatureName](features.SupportGateway),
 			SkipTests:         []string{},
 		})
 		if err != nil {
@@ -76,13 +75,14 @@ func TestMultipleGC(t *testing.T) {
 		privateGatewaySuiteGatewayClassName := "private"
 		privateGatewaySuite, err := suite.NewConformanceTestSuite(suite.ConformanceOptions{
 			Client:               c,
+			RestConfig:           cfg,
 			GatewayClassName:     privateGatewaySuiteGatewayClassName,
 			Debug:                *flags.ShowDebug,
 			CleanupBaseResources: *flags.CleanupBaseResources,
 			RunTest:              *flags.RunTest,
 			// SupportedFeatures cannot be empty, so we set it to SupportGateway
 			// All e2e tests should leave Features empty.
-			SupportedFeatures: sets.New[features.SupportedFeature](features.SupportGateway),
+			SupportedFeatures: sets.New[features.FeatureName](features.SupportGateway),
 			SkipTests:         []string{},
 		})
 		if err != nil {

test/e2e/tests/eg_upgrade.go

@@ -53,7 +53,7 @@ var EGUpgradeTest = suite.ConformanceTest{
 			depNS := "envoy-gateway-system"
 			lastVersionTag := os.Getenv("last_version_tag")
 			if lastVersionTag == "" {
-				lastVersionTag = "v1.0.2" // Default version tag if not specified
+				lastVersionTag = "v1.1.0" // Default version tag if not specified
 			}
 
 			// Uninstall the current version of EG
@@ -280,7 +280,7 @@ func migrateChartCRDs(actionConfig *action.Configuration, gatewayChart *chart.Ch
 				return err
 			}
 			// https://gateway-api.sigs.k8s.io/guides/?h=upgrade#v11-upgrade-notes
-			if newVersion == "v1.1.0" {
+			if newVersion == "v1.2.0-rc1" {
 				helper := resource.NewHelper(crd.Client, crd.Mapping)
 				existingCRD, err := helper.Get(crd.Namespace, crd.Name)
 				if kerrors.IsNotFound(err) {
@@ -292,8 +292,7 @@ func migrateChartCRDs(actionConfig *action.Configuration, gatewayChart *chart.Ch
 				if err != nil {
 					return err
 				}
-
-				if existingVersion == "v1.0.0" {
+				if existingVersion == "v1.1.0" {
 					// Delete the existing instance of the BTLS CRD
 					_, errs := actionConfig.KubeClient.Delete([]*resource.Info{crd})
 					if errs != nil {

test/e2e/upgrade/eg_upgrade_test.go

@@ -46,7 +46,7 @@ func TestEGUpgrade(t *testing.T) {
 		ManifestFS:           []fs.FS{e2e.UpgradeManifests},
 		RunTest:              *flags.RunTest,
 		BaseManifests:        "upgrade/manifests.yaml",
-		SupportedFeatures:    sets.New[features.SupportedFeature](features.SupportGateway),
+		SupportedFeatures:    sets.New[features.FeatureName](features.SupportGateway),
 		SkipTests:            []string{},
 	})
 	if err != nil {