Update ux to allow either "" or "core" for informer and rbac groups i…

…n annotations
kubernetes-sigs · May 18, 2018 · c65b397 · c65b397
1 parent 552cb2d
commit c65b397
Showing 1 changed file with 11 additions and 0 deletions.
diff --git a/cmd/internal/codegen/parse/rbac.go b/cmd/internal/codegen/parse/rbac.go
@@ -64,6 +64,14 @@ func parseRBACTag(tag string) rbacv1.PolicyRule {
 		values = strings.Split(value, ";")
 		switch kv[0] {
 		case "groups":
+			normalized := []string{}
+			for _, v := range values {
+				if v == "core" {
+					normalized = append(normalized, "")
+				} else {
+					normalized = append(normalized, v)
+				}
+			}
 			result.APIGroups = values
 		case "resources":
 			result.Resources = values
@@ -112,6 +120,9 @@ func parseInformerTag(tag string) v1.GroupVersionKind {
 		value := kv[1]
 		switch kv[0] {
 		case "group":
+			if value == "" {
+				value = "core"
+			}
 			result.Group = value
 		case "version":
 			result.Version = value