UPSTREAM: <carry>: warn only about unknown feature gates

OpenShift-Rebase-Source: a137009

Author: sttts

pkg/kubelet/apis/config/validation/validation_test.go

@@ -750,7 +750,9 @@ func TestValidateKubeletConfiguration(t *testing.T) {
 				conf.FeatureGates["invalid"] = true
 				return conf
 			},
-			errMsg: "unrecognized feature gate: invalid",
+			// In OpenShift we need to tolerate unrecognized feature gates
+			// errMsg: "unrecognized feature gate: invalid",
+			errMsg: "",
 		},
 	}
 

staging/src/k8s.io/component-base/compatibility/registry_test.go

@@ -265,7 +265,7 @@ func TestFlags(t *testing.T) {
 				"--emulated-version=test=2.7",
 				"--feature-gates=test:testD=true",
 			},
-			parseError: "unrecognized feature gate: testD",
+			// parseError: "unrecognized feature gate: testD",
 		},
 		{
 			name: "setting unknown component feature flag",

staging/src/k8s.io/component-base/featuregate/feature_gate.go

@@ -325,9 +325,8 @@ func (f *featureGate) unsafeSetFromMap(enabled map[Feature]bool, m map[string]bo
 		key := Feature(k)
 		versionedSpecs, ok := known[key]
 		if !ok {
-			// early return if encounters an unknown feature.
-			errs = append(errs, fmt.Errorf("unrecognized feature gate: %s", k))
-			return errs
+			klog.Warningf("unrecognized feature gate: %s", k)
+			continue
 		}
 		featureSpec := featureSpecAtEmulationVersion(versionedSpecs, emulationVersion)
 		if featureSpec.LockToDefault && featureSpec.Default != v {

staging/src/k8s.io/component-base/featuregate/feature_gate_test.go

@@ -88,7 +88,7 @@ func TestFeatureGateFlag(t *testing.T) {
 				testBetaGate:        false,
 				testLockedFalseGate: false,
 			},
-			parseError: "unrecognized feature gate: fooBarBaz",
+			//parseError: "unrecognized feature gate: fooBarBaz",
 		},
 		{
 			arg: "AllAlpha=false",
@@ -417,7 +417,7 @@ func TestFeatureGateSetFromMap(t *testing.T) {
 				testAlphaGate: false,
 				testBetaGate:  false,
 			},
-			setmapError: "unrecognized feature gate:",
+			//setmapError: "unrecognized feature gate:",
 		},
 		{
 			name: "set locked gates",
@@ -764,7 +764,7 @@ func TestVersionedFeatureGateFlag(t *testing.T) {
 				testAlphaGateNoVersion: false,
 				testBetaGateNoVersion:  false,
 			},
-			parseError: "unrecognized feature gate: fooBarBaz",
+			// parseError: "unrecognized feature gate: fooBarBaz",
 		},
 		{
 			arg: "AllAlpha=false",
@@ -1047,8 +1047,12 @@ func TestVersionedFeatureGateFlag(t *testing.T) {
 				errs = append(errs, err)
 			}
 			err = utilerrors.NewAggregate(errs)
+			strErr := ""
+			if err != nil {
+				strErr = err.Error()
+			}
 			if test.parseError != "" {
-				if !strings.Contains(err.Error(), test.parseError) {
+				if !strings.Contains(strErr, test.parseError) {
 					t.Errorf("%d: Parse() Expected %v, Got %v", i, test.parseError, err)
 				}
 				return
@@ -1591,9 +1595,9 @@ func TestCopyKnownFeatures(t *testing.T) {
 	require.NoError(t, fcopy.Set("FeatureB=false"))
 	assert.True(t, f.Enabled("FeatureB"))
 	assert.False(t, fcopy.Enabled("FeatureB"))
-	if err := fcopy.Set("FeatureC=true"); err == nil {
-		t.Error("expected FeatureC not registered in the copied feature gate")
-	}
+	// if err := fcopy.Set("FeatureC=true"); err == nil {
+	// 	t.Error("expected FeatureC not registered in the copied feature gate")
+	// }
 }
 
 func TestExplicitlySet(t *testing.T) {