Move responsive-gc feature to allowed

config/core/configmaps/features.yaml

@@ -20,7 +20,7 @@ metadata:
   labels:
     serving.knative.dev/release: devel
   annotations:
-    knative.dev/example-checksum: "7f1b7e2d"
+    knative.dev/example-checksum: "6a69cdef"
 data:
   _example: |
     ################################
@@ -105,10 +105,7 @@ data:
     # feature labels revisions in real-time as they become referenced and
     # dereferenced by Routes. This allows us to reap revisions shortly after
     # they are no longer active.
-    #
-    # ALPHA WARNING: This feature is not yet stable or complete. Enabling it
-    # should be used for testing purposes only.
-    responsive-revision-gc: "disabled"
+    responsive-revision-gc: "allowed"
 
     # Controls whether tag header based routing feature are enabled or not.
     # 1. Enabled: enabling tag header based routing

pkg/apis/config/features.go

@@ -48,7 +48,7 @@ func defaultFeaturesConfig() *Features {
 		PodSpecRuntimeClassName: Disabled,
 		PodSpecSecurityContext:  Disabled,
 		PodSpecTolerations:      Disabled,
-		ResponsiveRevisionGC:    Disabled,
+		ResponsiveRevisionGC:    Allowed,
 		TagHeaderBasedRouting:   Disabled,
 	}
 }

pkg/reconciler/configuration/resources/revision_test.go

@@ -323,9 +323,7 @@ func TestMakeRevisions(t *testing.T) {
 	for _, test := range tests {
 		t.Run(test.name, func(t *testing.T) {
 			ctx := context.Background()
-			if test.responsiveGC {
-				ctx = enableResponsiveGC(ctx)
-			}
+			ctx = enableResponsiveGC(ctx, test.responsiveGC)
 
 			got := MakeRevision(ctx, test.configuration, clock)
 			if diff := cmp.Diff(test.want, got); diff != "" {
@@ -335,11 +333,16 @@ func TestMakeRevisions(t *testing.T) {
 	}
 }
 
-func enableResponsiveGC(ctx context.Context) context.Context {
+func enableResponsiveGC(ctx context.Context, enabled bool) context.Context {
+	flag := cfgmap.Disabled
+	if enabled {
+		flag = cfgmap.Enabled
+	}
+
 	defaultDefaults, _ := cfgmap.NewDefaultsConfigFromMap(map[string]string{})
 	c := &config.Config{
 		Features: &cfgmap.Features{
-			ResponsiveRevisionGC: cfgmap.Enabled,
+			ResponsiveRevisionGC: flag,
 		},
 		Defaults: defaultDefaults,
 	}

pkg/reconciler/service/resources/configuration.go

@@ -32,7 +32,7 @@ import (
 
 // MakeConfiguration creates a Configuration from a Service object.
 func MakeConfiguration(service *v1.Service) (*v1.Configuration, error) {
-	return MakeConfigurationFromExisting(service, &v1.Configuration{}, cfgmap.Disabled)
+	return MakeConfigurationFromExisting(service, &v1.Configuration{}, cfgmap.Allowed)
 }
 
 // MakeConfigurationFromExisting creates a Configuration from a Service object given an existing Configuration.