From 423fd341f1576d4785c72a980499aeb3ce2ac120 Mon Sep 17 00:00:00 2001 From: Justin Kulikauskas Date: Mon, 5 Dec 2022 16:52:16 -0500 Subject: [PATCH] Remove confusing Undetermined compliance state The situation where this compliance state was being used was appearing if the ConfigurationPolicy had no templates. Marking it as Unknown is better, since at least that is a value listed in the api folder... This change also makes this situation create a compliance event, since the framework *needs* a new event in all situations when a policy, else when it transitions from Pending to something else, it will not update. Refs: - https://issues.redhat.com/browse/ACM-2263 Signed-off-by: Justin Kulikauskas --- controllers/configurationpolicy_controller.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/controllers/configurationpolicy_controller.go b/controllers/configurationpolicy_controller.go index e41aa005..3abeaeba 100644 --- a/controllers/configurationpolicy_controller.go +++ b/controllers/configurationpolicy_controller.go @@ -2407,7 +2407,7 @@ func (r *ConfigurationPolicyReconciler) addForUpdate(policy *policyv1.Configurat if policy.ObjectMeta.DeletionTimestamp != nil { policy.Status.ComplianceState = policyv1.Terminating } else if len(policy.Status.CompliancyDetails) == 0 { - policy.Status.ComplianceState = "Undetermined" + policy.Status.ComplianceState = policyv1.UnknownCompliancy } else if compliant { policy.Status.ComplianceState = policyv1.Compliant } else { @@ -2443,7 +2443,7 @@ func (r *ConfigurationPolicyReconciler) updatePolicyStatus( policy *policyv1.ConfigurationPolicy, sendEvent bool, ) (*policyv1.ConfigurationPolicy, error) { - if policy.Status.ComplianceState != "Undetermined" && sendEvent { + if sendEvent { log.V(1).Info("Sending parent policy compliance event") // If the compliance event can't be created, then don't update the ConfigurationPolicy @@ -2516,7 +2516,7 @@ func (r *ConfigurationPolicyReconciler) sendComplianceEvent(instance *policyv1.C ReportingInstance: r.InstanceName, } - if instance.Status.ComplianceState == policyv1.NonCompliant { + if instance.Status.ComplianceState != policyv1.Compliant { event.Type = "Warning" } @@ -2525,8 +2525,8 @@ func (r *ConfigurationPolicyReconciler) sendComplianceEvent(instance *policyv1.C // convertPolicyStatusToString to be able to pass the status as event func convertPolicyStatusToString(plc *policyv1.ConfigurationPolicy) (results string) { - if plc.Status.ComplianceState == "" { - return "ComplianceState is still undetermined" + if plc.Status.ComplianceState == "" || plc.Status.ComplianceState == policyv1.UnknownCompliancy { + return "ComplianceState is still unknown" } result := string(plc.Status.ComplianceState)