Merge pull request #1754 from chrischdi/pr-customresource-underscore-fix

k8s-ci-robot · web-flow · commit aa8729abea95 · 2022-06-09T04:17:42.000-07:00
fix customresourcestate metric names to not contain underscores
diff --git a/pkg/customresourcestate/config.go b/pkg/customresourcestate/config.go
@@ -132,6 +132,7 @@ func (r Resource) GetSubsystem() string {
 		return strings.NewReplacer(
 			"/", "_",
 			".", "_",
+			"-", "_",
 		).Replace(fmt.Sprintf("%s_%s_%s", r.GroupVersionKind.Group, r.GroupVersionKind.Version, r.GroupVersionKind.Kind))
 	}
 	if r.Subsystem == "_" {

Original file line number	Diff line number	Diff line change
`@@ -132,6 +132,7 @@ func (r Resource) GetSubsystem() string {`
`132`	`132`	`return strings.NewReplacer(`
`133`	`133`	`"/", "_",`
`134`	`134`	`".", "_",`
	`135`	`+ "-", "_",`
`135`	`136`	`).Replace(fmt.Sprintf("%s_%s_%s", r.GroupVersionKind.Group, r.GroupVersionKind.Version, r.GroupVersionKind.Kind))`
`136`	`137`	`}`
`137`	`138`	`if r.Subsystem == "_" {`