Implementation: use EndpointSlices by default

apis/projectcontour/v1alpha1/contourconfig.go

@@ -88,8 +88,8 @@ type ContourConfigurationSpec struct {
 	// FeatureFlags defines toggle to enable new contour features.
 	// Available toggles are:
 	// useEndpointSlices - configures contour to fetch endpoint data
-	// from k8s endpoint slices. defaults to false and reading endpoint
-	// data from the k8s endpoints.
+	// from k8s endpoint slices. defaults to true.
+	// if false then reading endpoint data from the k8s endpoints.
 	FeatureFlags FeatureFlags `json:"featureFlags,omitempty"`
 }
 

apis/projectcontour/v1alpha1/contourconfig_helpers.go

@@ -15,18 +15,16 @@ package v1alpha1
 
 import (
 	"fmt"
-	"slices"
 	"strconv"
+	"strings"
 
 	"k8s.io/apimachinery/pkg/util/sets"
 )
 
-const (
-	featureFlagUseEndpointSlices string = "useEndpointSlices"
-)
+const featureFlagUseEndpointSlices string = "useEndpointSlices"
 
-var featureFlagsMap = map[string]bool{
-	featureFlagUseEndpointSlices: true,
+var featureFlagsMap = map[string]struct{}{
+	featureFlagUseEndpointSlices: {},
 }
 
 // Validate configuration that is not already covered by CRD validation.
@@ -226,16 +224,26 @@ func (e *EnvoyTLS) SanitizedCipherSuites() []string {
 
 func (f FeatureFlags) Validate() error {
 	for _, featureFlag := range f {
-		if _, found := featureFlagsMap[featureFlag]; !found {
+		fields := strings.Split(featureFlag, "=")
+		if _, found := featureFlagsMap[fields[0]]; !found {
 			return fmt.Errorf("invalid contour configuration, unknown feature flag:%s", featureFlag)
 		}
 	}
-
 	return nil
 }
 
 func (f FeatureFlags) IsEndpointSliceEnabled() bool {
-	return slices.Contains(f, featureFlagUseEndpointSlices)
+	// only when the flag: 'useEndpointSlices=false' is exists, return false
+	for _, flag := range f {
+		if !strings.HasPrefix(flag, featureFlagUseEndpointSlices) {
+			continue
+		}
+		fields := strings.Split(flag, "=")
+		if len(fields) == 2 && strings.ToLower(fields[1]) == "false" {
+			return false
+		}
+	}
+	return true
 }
 
 // Validate ensures that exactly one of ControllerName or GatewayRef are specified.

apis/projectcontour/v1alpha1/contourconfig_helpers_test.go

@@ -302,10 +302,26 @@ func TestFeatureFlagsValidate(t *testing.T) {
 		expected error
 	}{
 		{
-			name:     "valid flag",
+			name:     "valid flag: no value",
 			flags:    v1alpha1.FeatureFlags{"useEndpointSlices"},
 			expected: nil,
 		},
+		{
+			name:     "valid flag2: empty",
+			flags:    v1alpha1.FeatureFlags{"useEndpointSlices="},
+			expected: nil,
+		},
+		{
+			name:     "valid flag: true",
+			flags:    v1alpha1.FeatureFlags{"useEndpointSlices=true"},
+			expected: nil,
+		},
+		{
+			name:     "valid flag: false",
+			flags:    v1alpha1.FeatureFlags{"useEndpointSlices=false"},
+			expected: nil,
+		},
+
 		{
 			name:     "invalid flag",
 			flags:    v1alpha1.FeatureFlags{"invalidFlag"},
@@ -330,3 +346,68 @@ func TestFeatureFlagsValidate(t *testing.T) {
 		})
 	}
 }
+
+func TestFeatureFlagsIsEndpointSliceEnabled(t *testing.T) {
+	tests := []struct {
+		name     string
+		flags    v1alpha1.FeatureFlags
+		expected bool
+	}{
+		{
+			name:     "valid flag: no value",
+			flags:    v1alpha1.FeatureFlags{"useEndpointSlices"},
+			expected: true,
+		},
+		{
+			name:     "valid flag2: empty",
+			flags:    v1alpha1.FeatureFlags{"useEndpointSlices="},
+			expected: true,
+		},
+		{
+			name:     "valid flag: true",
+			flags:    v1alpha1.FeatureFlags{"useEndpointSlices=true"},
+			expected: true,
+		},
+		{
+			name:     "valid flag: ANY",
+			flags:    v1alpha1.FeatureFlags{"useEndpointSlices=ANY"},
+			expected: true,
+		},
+
+		{
+			name:     "empty flags",
+			flags:    v1alpha1.FeatureFlags{},
+			expected: true,
+		},
+		{
+			name:     "empty string",
+			flags:    v1alpha1.FeatureFlags{""},
+			expected: true,
+		},
+
+		{
+			name:     "multi-flags",
+			flags:    v1alpha1.FeatureFlags{"useEndpointSlices", "otherFlag"},
+			expected: true,
+		},
+
+		{
+			name:     "valid flag: false",
+			flags:    v1alpha1.FeatureFlags{"useEndpointSlices=false"},
+			expected: false,
+		},
+
+		{
+			name:     "valid flag: FALSE",
+			flags:    v1alpha1.FeatureFlags{"useEndpointSlices=FALSE"},
+			expected: false,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			err := tt.flags.IsEndpointSliceEnabled()
+			assert.Equal(t, tt.expected, err)
+		})
+	}
+}

changelogs/unreleased/6149-izturn-minor.md

@@ -0,0 +1,5 @@
+
+## Use EndpointSlices by default
+
+Enables contour to consume the kubernetes endpointslice API to determine the endpoints to configure Envoy with by default.
+Note: if you need to continue using the old configuration, please set the flag: `useEndpointSlices=false` explicitly.

examples/contour/01-crds.yaml

@@ -601,8 +601,8 @@ spec:
                   FeatureFlags defines toggle to enable new contour features.
                   Available toggles are:
                   useEndpointSlices - configures contour to fetch endpoint data
-                  from k8s endpoint slices. defaults to false and reading endpoint
-                  data from the k8s endpoints.
+                  from k8s endpoint slices. defaults to true.
+                  if false then reading endpoint data from the k8s endpoints.
                 items:
                   type: string
                 type: array
@@ -4282,8 +4282,8 @@ spec:
                       FeatureFlags defines toggle to enable new contour features.
                       Available toggles are:
                       useEndpointSlices - configures contour to fetch endpoint data
-                      from k8s endpoint slices. defaults to false and reading endpoint
-                      data from the k8s endpoints.
+                      from k8s endpoint slices. defaults to true.
+                      if false then reading endpoint data from the k8s endpoints.
                     items:
                       type: string
                     type: array

examples/render/contour-deployment.yaml

@@ -820,8 +820,8 @@ spec:
                   FeatureFlags defines toggle to enable new contour features.
                   Available toggles are:
                   useEndpointSlices - configures contour to fetch endpoint data
-                  from k8s endpoint slices. defaults to false and reading endpoint
-                  data from the k8s endpoints.
+                  from k8s endpoint slices. defaults to true.
+                  if false then reading endpoint data from the k8s endpoints.
                 items:
                   type: string
                 type: array
@@ -4501,8 +4501,8 @@ spec:
                       FeatureFlags defines toggle to enable new contour features.
                       Available toggles are:
                       useEndpointSlices - configures contour to fetch endpoint data
-                      from k8s endpoint slices. defaults to false and reading endpoint
-                      data from the k8s endpoints.
+                      from k8s endpoint slices. defaults to true.
+                      if false then reading endpoint data from the k8s endpoints.
                     items:
                       type: string
                     type: array

examples/render/contour-gateway-provisioner.yaml

@@ -612,8 +612,8 @@ spec:
                   FeatureFlags defines toggle to enable new contour features.
                   Available toggles are:
                   useEndpointSlices - configures contour to fetch endpoint data
-                  from k8s endpoint slices. defaults to false and reading endpoint
-                  data from the k8s endpoints.
+                  from k8s endpoint slices. defaults to true.
+                  if false then reading endpoint data from the k8s endpoints.
                 items:
                   type: string
                 type: array
@@ -4293,8 +4293,8 @@ spec:
                       FeatureFlags defines toggle to enable new contour features.
                       Available toggles are:
                       useEndpointSlices - configures contour to fetch endpoint data
-                      from k8s endpoint slices. defaults to false and reading endpoint
-                      data from the k8s endpoints.
+                      from k8s endpoint slices. defaults to true.
+                      if false then reading endpoint data from the k8s endpoints.
                     items:
                       type: string
                     type: array

examples/render/contour-gateway.yaml

@@ -823,8 +823,8 @@ spec:
                   FeatureFlags defines toggle to enable new contour features.
                   Available toggles are:
                   useEndpointSlices - configures contour to fetch endpoint data
-                  from k8s endpoint slices. defaults to false and reading endpoint
-                  data from the k8s endpoints.
+                  from k8s endpoint slices. defaults to true.
+                  if false then reading endpoint data from the k8s endpoints.
                 items:
                   type: string
                 type: array
@@ -4504,8 +4504,8 @@ spec:
                       FeatureFlags defines toggle to enable new contour features.
                       Available toggles are:
                       useEndpointSlices - configures contour to fetch endpoint data
-                      from k8s endpoint slices. defaults to false and reading endpoint
-                      data from the k8s endpoints.
+                      from k8s endpoint slices. defaults to true.
+                      if false then reading endpoint data from the k8s endpoints.
                     items:
                       type: string
                     type: array

examples/render/contour.yaml

@@ -820,8 +820,8 @@ spec:
                   FeatureFlags defines toggle to enable new contour features.
                   Available toggles are:
                   useEndpointSlices - configures contour to fetch endpoint data
-                  from k8s endpoint slices. defaults to false and reading endpoint
-                  data from the k8s endpoints.
+                  from k8s endpoint slices. defaults to true.
+                  if false then reading endpoint data from the k8s endpoints.
                 items:
                   type: string
                 type: array
@@ -4501,8 +4501,8 @@ spec:
                       FeatureFlags defines toggle to enable new contour features.
                       Available toggles are:
                       useEndpointSlices - configures contour to fetch endpoint data
-                      from k8s endpoint slices. defaults to false and reading endpoint
-                      data from the k8s endpoints.
+                      from k8s endpoint slices. defaults to true.
+                      if false then reading endpoint data from the k8s endpoints.
                     items:
                       type: string
                     type: array

pkg/config/parameters.go

@@ -728,8 +728,8 @@ type Parameters struct {
 	// FeatureFlags defines toggle to enable new contour features.
 	// available toggles are
 	// useEndpointSlices - configures contour to fetch endpoint data
-	// from k8s endpoint slices. defaults to false and reading endpoint
-	// data from the k8s endpoints.
+	// from k8s endpoint slices. defaults to true,
+	// if false then reading endpoint data from the k8s endpoints.
 	FeatureFlags []string `yaml:"featureFlags,omitempty"`
 }
 

site/content/docs/main/config/api-reference.html

@@ -5184,8 +5184,8 @@ <h3 id="projectcontour.io/v1alpha1.ContourConfiguration">ContourConfiguration
 <p>FeatureFlags defines toggle to enable new contour features.
 Available toggles are:
 useEndpointSlices - configures contour to fetch endpoint data
-from k8s endpoint slices. defaults to false and reading endpoint
-data from the k8s endpoints.</p>
+from k8s endpoint slices. defaults to true.
+if false then reading endpoint data from the k8s endpoints.</p>
 </td>
 </tr>
 </table>
@@ -5977,8 +5977,8 @@ <h3 id="projectcontour.io/v1alpha1.ContourConfigurationSpec">ContourConfiguratio
 <p>FeatureFlags defines toggle to enable new contour features.
 Available toggles are:
 useEndpointSlices - configures contour to fetch endpoint data
-from k8s endpoint slices. defaults to false and reading endpoint
-data from the k8s endpoints.</p>
+from k8s endpoint slices. defaults to true.
+if false then reading endpoint data from the k8s endpoints.</p>
 </td>
 </tr>
 </tbody>