Merge pull request #1809 from chrischdi/pr-info-nometric-nil

Do not expose info metric for nil objects

Author: k8s-ci-robot

pkg/customresourcestate/registry_factory.go

@@ -269,6 +269,9 @@ func (c *compiledInfo) Values(v interface{}) (result []eachValue, errs []error)
 }
 
 func (c *compiledInfo) values(v interface{}) (result []eachValue, err []error) {
+	if v == nil {
+		return
+	}
 	value := eachValue{Value: 1, Labels: map[string]string{}}
 	addPathLabels(v, c.labelFromPath, value.Labels)
 	result = append(result, value)

pkg/customresourcestate/registry_factory_test.go

@@ -209,6 +209,11 @@ func Test_values(t *testing.T) {
 		}, wantResult: []eachValue{
 			newEachValue(t, 1, "version", "v0.0.0"),
 		}},
+		{name: "info nil path", each: &compiledInfo{
+			compiledCommon{
+				path: mustCompilePath(t, "does", "not", "exist"),
+			},
+		}, wantResult: nil},
 		{name: "stateset", each: &compiledStateSet{
 			compiledCommon: compiledCommon{
 				path: mustCompilePath(t, "status", "phase"),