rename metric and add feature stage to metrics

Change-Id: Iccab766ba2ee16ef54abb79e41ee61a6117fa7dd

staging/src/k8s.io/component-base/metrics/prometheus/feature/metrics.go

@@ -25,27 +25,26 @@ import (
 )
 
 var (
-	// featureEnabled is a Prometheus Gauge metrics used for recording the enablement of a k8s feature.
+	// featureInfo is a Prometheus Gauge metrics used for recording the enablement of a k8s feature.
-	featureEnabled = k8smetrics.NewGaugeVec(
+	featureInfo = k8smetrics.NewGaugeVec(
 		&k8smetrics.GaugeOpts{
 			Namespace:      "k8s",
-			Name:           "feature_enabled",
+			Name:           "feature_info",
-			Help:           "This metric records the result of whether a feature is enabled.",
+			Help:           "This metric records the data about the stage and enablement of a k8s feature.",
 			StabilityLevel: k8smetrics.ALPHA,
 		},
-		[]string{"name", "enabled"},
+		[]string{"name", "stage", "enabled"},
 	)
 )
 
 func init() {
-	legacyregistry.MustRegister(featureEnabled)
+	legacyregistry.MustRegister(featureInfo)
 }
 
-func ResetFeatureEnabledMetric() {
+func ResetFeatureInfoMetric() {
-	featureEnabled.Reset()
+	featureInfo.Reset()
 }
 
-func RecordFeatureEnabled(ctx context.Context, name string, enabled bool) error {
+func RecordFeatureInfo(ctx context.Context, name string, stage string, enabled bool) {
-	featureEnabled.WithContext(ctx).WithLabelValues(name, fmt.Sprintf("%v", enabled)).Set(1)
+	featureInfo.WithContext(ctx).WithLabelValues(name, stage, fmt.Sprintf("%v", enabled)).Set(1)
-	return nil
 }

staging/src/k8s.io/component-base/metrics/prometheus/feature/metrics_test.go

@@ -26,48 +26,48 @@ import (
 )
 
 var (
-	testedMetrics = []string{"k8s_feature_enabled"}
+	testedMetrics = []string{"k8s_feature_info"}
 )
 
 func TestObserveHealthcheck(t *testing.T) {
 	defer legacyregistry.Reset()
-	defer ResetFeatureEnabledMetric()
+	defer ResetFeatureInfoMetric()
 
 	testCases := []struct {
 		desc    string
 		name    string
+		stage   string
 		enabled bool
 		want    string
 	}{
 		{
 			desc:    "test enabled",
 			name:    "feature-a",
+			stage:   "ALPHA",
 			enabled: true,
 			want: `
-       	# HELP k8s_feature_enabled [ALPHA] This metric records the result of whether a feature is enabled.
+       	# HELP k8s_feature_info [ALPHA] This metric records the data about the stage and enablement of a k8s feature.
-        # TYPE k8s_feature_enabled gauge
+        # TYPE k8s_feature_info gauge
-        k8s_feature_enabled{enabled="true",name="feature-a"} 1
+        k8s_feature_info{enabled="true",name="feature-a",stage="ALPHA"} 1
 `,
 		},
 		{
 			desc:    "test disabled",
 			name:    "feature-b",
+			stage:   "BETA",
 			enabled: false,
 			want: `
-       	# HELP k8s_feature_enabled [ALPHA] This metric records the result of whether a feature is enabled.
+       	# HELP k8s_feature_info [ALPHA] This metric records the data about the stage and enablement of a k8s feature.
-        # TYPE k8s_feature_enabled gauge
+        # TYPE k8s_feature_info gauge
-        k8s_feature_enabled{enabled="false",name="feature-b"} 1
+        k8s_feature_info{enabled="false",name="feature-b",stage="BETA"} 1
 `,
 		},
 	}
 
 	for _, test := range testCases {
 		t.Run(test.desc, func(t *testing.T) {
-			defer ResetFeatureEnabledMetric()
+			defer ResetFeatureInfoMetric()
-			err := RecordFeatureEnabled(context.Background(), test.name, test.enabled)
+			RecordFeatureInfo(context.Background(), test.name, test.stage, test.enabled)
-			if err != nil {
-				t.Errorf("unexpected err: %v", err)
-			}
 
 			if err := testutil.GatherAndCompare(legacyregistry.DefaultGatherer, strings.NewReader(test.want), testedMetrics...); err != nil {
 				t.Fatal(err)