Fix RegEx to allow any character in a template name

If an invalid character is provided in a template name, our RegEx was
filtering it out and not handling it properly.

ref: https://issues.redhat.com/browse/ACM-7737
Signed-off-by: Dale Haiducek <[email protected]>

Author: dhaiducek

controllers/statussync/policy_status_sync.go

@@ -184,7 +184,7 @@ func (r *PolicyReconciler) Reconcile(ctx context.Context, request reconcile.Requ
 	// filter events to current policy instance and build map
 	eventForPolicyMap := make(map[string]*[]historyEvent)
 	// panic if regexp invalid
-	rgx := regexp.MustCompile(`(?i)^policy:\s*(?:([a-z0-9.-]+)\s*\/)?([a-z0-9.-]+)`)
+	rgx := regexp.MustCompile(`(?i)^policy:\s*(?:([a-z0-9.-]+)\s*\/)?(.+)`)
 	for _, event := range eventList.Items {
 		// sample event.Reason -- reason: 'policy: calamari/policy-grc-rbactest-example'
 		reason := rgx.FindString(event.Reason)

test/e2e/case10_error_test.go

@@ -11,7 +11,9 @@ import (
 	. "github.com/onsi/gomega"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
+	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/types"
+	policiesv1 "open-cluster-management.io/governance-policy-propagator/api/v1"
 	"open-cluster-management.io/governance-policy-propagator/test/utils"
 )
 
@@ -110,6 +112,35 @@ var _ = Describe("Test error handling", func() {
 			1,
 		).Should(BeTrue())
 	})
+	It("should generate creation err event", func() {
+		policyName := "case10-invalid-name-error"
+		statusMsg := "template-error; Failed to create policy template:"
+
+		hubApplyPolicy(policyName,
+			yamlBasePath+"invalid-name-error.yaml")
+
+		By("Checking for event with creation err on managed cluster in ns:" + clusterNamespace)
+		Eventually(
+			checkForEvent(policyName, statusMsg),
+			defaultTimeoutSeconds,
+			1,
+		).Should(BeTrue())
+		By("Checking if policy status is noncompliant")
+		hubPlc := utils.GetWithTimeout(
+			clientHubDynamic,
+			gvrPolicy,
+			policyName,
+			clusterNamespaceOnHub,
+			true,
+			defaultTimeoutSeconds)
+		var plc *policiesv1.Policy
+		err := runtime.DefaultUnstructuredConverter.FromUnstructured(hubPlc.Object, &plc)
+		Expect(err).ToNot(HaveOccurred())
+		Expect(plc.Status.Details).To(HaveLen(1))
+		Expect(plc.Status.Details[0].History).To(HaveLen(1))
+		Expect(plc.Status.Details[0].TemplateMeta.GetName()).To(Equal("case10_invalid-name"))
+		Expect(plc.Status.Details[0].History[0].Message).To(ContainSubstring(statusMsg))
+	})
 	It("should generate unsupported object err event", func() {
 		hubApplyPolicy("case10-unsupported-object",
 			yamlBasePath+"unsupported-object-error.yaml")

test/resources/case10_template_sync_error_test/invalid-name-error.yaml

@@ -0,0 +1,30 @@
+apiVersion: policy.open-cluster-management.io/v1
+kind: Policy
+metadata:
+  name: case10-invalid-name-error
+  labels:
+    policy.open-cluster-management.io/cluster-name: managed
+    policy.open-cluster-management.io/cluster-namespace: managed
+    policy.open-cluster-management.io/root-policy: case10-template-name-error
+spec:
+  remediationAction: inform
+  disabled: false
+  policy-templates:
+    - objectDefinition:
+        apiVersion: policy.open-cluster-management.io/v1
+        kind: ConfigurationPolicy
+        metadata:
+          name: case10_invalid-name
+        spec:
+          object-templates:
+            - complianceType: musthave
+              objectDefinition:
+                apiVersion: v1
+                kind: Pod
+                metadata:
+                  name: nginx-pod-e2e
+                  namespace: default
+                spec:
+                  containers:
+                    - name: nginx
+