Merge pull request #3837 from camilamacedo86/fix-path-core-types

🐛 fix resource path generation for resources without a specified group

Author: k8s-ci-robot

pkg/plugins/common/kustomize/v2/scaffolds/internal/templates/config/crd/kustomization.go

@@ -85,7 +85,7 @@ func (f *Kustomization) GetCodeFragments() machinery.CodeFragmentsMap {
 	res = append(res, fmt.Sprintf(resourceCodeFragment, f.Resource.QualifiedGroup(), f.Resource.Plural))
 
 	suffix := f.Resource.Plural
-	if f.MultiGroup {
+	if f.MultiGroup && f.Resource.Group != "" {
 		suffix = f.Resource.Group + "_" + f.Resource.Plural
 	}
 	// Generate resource code fragments

pkg/plugins/common/kustomize/v2/scaffolds/internal/templates/config/crd/patches/enablecainjection_patch.go

@@ -34,7 +34,7 @@ type EnableCAInjectionPatch struct {
 // SetTemplateDefaults implements file.Template
 func (f *EnableCAInjectionPatch) SetTemplateDefaults() error {
 	if f.Path == "" {
-		if f.MultiGroup {
+		if f.MultiGroup && f.Resource.Group != "" {
 			f.Path = filepath.Join("config", "crd", "patches", "cainjection_in_%[group]_%[plural].yaml")
 		} else {
 			f.Path = filepath.Join("config", "crd", "patches", "cainjection_in_%[plural].yaml")

pkg/plugins/common/kustomize/v2/scaffolds/internal/templates/config/crd/patches/enablewebhook_patch.go

@@ -34,7 +34,7 @@ type EnableWebhookPatch struct {
 // SetTemplateDefaults implements file.Template
 func (f *EnableWebhookPatch) SetTemplateDefaults() error {
 	if f.Path == "" {
-		if f.MultiGroup {
+		if f.MultiGroup && f.Resource.Group != "" {
 			f.Path = filepath.Join("config", "crd", "patches", "webhook_in_%[group]_%[plural].yaml")
 		} else {
 			f.Path = filepath.Join("config", "crd", "patches", "webhook_in_%[plural].yaml")

pkg/plugins/common/kustomize/v2/scaffolds/internal/templates/config/rbac/crd_editor_role.go

@@ -35,7 +35,7 @@ type CRDEditorRole struct {
 // SetTemplateDefaults implements file.Template
 func (f *CRDEditorRole) SetTemplateDefaults() error {
 	if f.Path == "" {
-		if f.MultiGroup {
+		if f.MultiGroup && f.Resource.Group != "" {
 			f.Path = filepath.Join("config", "rbac", "%[group]_%[kind]_editor_role.yaml")
 		} else {
 			f.Path = filepath.Join("config", "rbac", "%[kind]_editor_role.yaml")

pkg/plugins/common/kustomize/v2/scaffolds/internal/templates/config/rbac/crd_viewer_role.go

@@ -35,7 +35,7 @@ type CRDViewerRole struct {
 // SetTemplateDefaults implements file.Template
 func (f *CRDViewerRole) SetTemplateDefaults() error {
 	if f.Path == "" {
-		if f.MultiGroup {
+		if f.MultiGroup && f.Resource.Group != "" {
 			f.Path = filepath.Join("config", "rbac", "%[group]_%[kind]_viewer_role.yaml")
 		} else {
 			f.Path = filepath.Join("config", "rbac", "%[kind]_viewer_role.yaml")

pkg/plugins/common/kustomize/v2/scaffolds/internal/templates/config/samples/crd_sample.go

@@ -36,7 +36,11 @@ type CRDSample struct {
 // SetTemplateDefaults implements file.Template
 func (f *CRDSample) SetTemplateDefaults() error {
 	if f.Path == "" {
-		f.Path = filepath.Join("config", "samples", "%[group]_%[version]_%[kind].yaml")
+		if f.Resource.Group != "" {
+			f.Path = filepath.Join("config", "samples", "%[group]_%[version]_%[kind].yaml")
+		} else {
+			f.Path = filepath.Join("config", "samples", "%[version]_%[kind].yaml")
+		}
 	}
 	f.Path = f.Resource.Replacer().Replace(f.Path)
 

pkg/plugins/common/kustomize/v2/scaffolds/internal/templates/config/samples/kustomization.go

@@ -59,7 +59,11 @@ const samplesCodeFragment = `- %s
 // makeCRFileName returns a Custom Resource example file name in the same format
 // as kubebuilder's CreateAPI plugin for a gvk.
 func (f Kustomization) makeCRFileName() string {
-	return f.Resource.Replacer().Replace("%[group]_%[version]_%[kind].yaml")
+	if f.Resource.Group != "" {
+		return f.Resource.Replacer().Replace("%[group]_%[version]_%[kind].yaml")
+	}
+	return f.Resource.Replacer().Replace("%[version]_%[kind].yaml")
+
 }
 
 // GetCodeFragments implements file.Inserter

pkg/plugins/golang/deploy-image/v1alpha1/scaffolds/internal/templates/config/samples/crd_sample.go

@@ -35,7 +35,11 @@ type CRDSample struct {
 // SetTemplateDefaults implements file.Template
 func (f *CRDSample) SetTemplateDefaults() error {
 	if f.Path == "" {
-		f.Path = filepath.Join("config", "samples", "%[group]_%[version]_%[kind].yaml")
+		if f.Resource.Group != "" {
+			f.Path = filepath.Join("config", "samples", "%[group]_%[version]_%[kind].yaml")
+		} else {
+			f.Path = filepath.Join("config", "samples", "%[version]_%[kind].yaml")
+		}
 	}
 	f.Path = f.Resource.Replacer().Replace(f.Path)
 	log.Println(f.Path)

testdata/project-v4-multigroup-with-deploy-image/config/crd/kustomization.yaml

@@ -26,7 +26,7 @@ patches:
 #- path: patches/webhook_in_foo.policy_healthcheckpolicies.yaml
 #- path: patches/webhook_in_foo_bars.yaml
 #- path: patches/webhook_in_fiz_bars.yaml
-#- path: patches/webhook_in__lakers.yaml
+#- path: patches/webhook_in_lakers.yaml
 #+kubebuilder:scaffold:crdkustomizewebhookpatch
 
 # [CERTMANAGER] To enable cert-manager, uncomment all the sections with [CERTMANAGER] prefix.
@@ -40,7 +40,7 @@ patches:
 #- path: patches/cainjection_in_foo.policy_healthcheckpolicies.yaml
 #- path: patches/cainjection_in_foo_bars.yaml
 #- path: patches/cainjection_in_fiz_bars.yaml
-#- path: patches/cainjection_in__lakers.yaml
+#- path: patches/cainjection_in_lakers.yaml
 #+kubebuilder:scaffold:crdkustomizecainjectionpatch
 
 # [WEBHOOK] To enable webhook, uncomment the following section