dale comment

Signed-off-by: Yi Rae Kim <yikim@redhat.com>

controllers/configurationpolicy_controller.go

@@ -1114,20 +1114,21 @@ func (r *ConfigurationPolicyReconciler) handleObjectTemplates(plc policyv1.Confi
 			}
 
 			if mergeMessageEnforce {
-				names = append([]string{}, templateObjs[indx].name)
+				names = []string{templateObjs[indx].name}
 			}
 
 			// object template enforced, already handled in handleObjects
 			if names == nil {
 				handled = true
 			} else {
+				// when multiple messages and enforce mode, it doesn't mean remediactionAction is changed
 				enforce = false
 			}
 
 			// violations for enforce configurationpolicies are already handled in handleObjects,
 			// so we only need to generate a violation if the remediationAction is set to inform
 			// Or multiple namespaces and enforce use this
-			if mergeMessageEnforce || (!handled && !enforce) {
+			if !handled && !enforce {
 				if !compliant {
 					if len(names) == 0 {
 						numNonCompliant++
@@ -1153,7 +1154,7 @@ func (r *ConfigurationPolicyReconciler) handleObjectTemplates(plc policyv1.Confi
 			}
 		}
 
-		if mergeMessageEnforce || (!handled && !enforce) {
+		if !handled && !enforce {
 			objData := templateIdentifier{
 				index:       indx,
 				kind:        kind,
@@ -1169,10 +1170,6 @@ func (r *ConfigurationPolicyReconciler) handleObjectTemplates(plc policyv1.Confi
 				parentStatusUpdateNeeded = true
 			}
 
-			if mergeMessageEnforce {
-				r.Recorder.Event(&plc, eventNormal, fmt.Sprintf(plcFmtStr, plc.GetName()),
-					convertPolicyStatusToString(&plc))
-			}
 		}
 	}
 
@@ -1362,7 +1359,7 @@ type templateIdentifier struct {
 	namespaced  bool
 }
 
-// createInformStatus updates the status field for a configurationpolicy with remediationAction=inform
+// createMergedStatus updates the status field for a configurationpolicy with remediationAction=inform
 // based on how many compliant/noncompliant objects are found when processing the templates in the configurationpolicy
 // Or multiple namespaces and enforce use this
 func createMergedStatus(