Skip to content

Commit

Permalink
fix some pending policy (stolostron#893)
Browse files Browse the repository at this point in the history
Signed-off-by: ldpliu <[email protected]>
  • Loading branch information
ldpliu authored Apr 23, 2024
1 parent f3a5026 commit 112b353
Show file tree
Hide file tree
Showing 8 changed files with 839 additions and 821 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ func policyEventPredicate(ctx context.Context, obj client.Object, c client.Clien
// get policy
policy, err := getInvolvePolicy(ctx, c, evt)
if err != nil {
log.Error(err, "failed to get involved policy", "event", evt.Namespace+"/"+evt.Name)
log.Info("failed to get involved policy", "event", evt.Namespace+"/"+evt.Name, "error", err.Error())
return nil, false
}
return policy, true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ func handleCompliance(log logr.Logger, ctx context.Context, evt *cloudevents.Eve

// handle pending compliance clusters of the policy
pendingCompliances := newLocalCompliances(leafHub, policyID, database.Pending,
eventCompliance.PendingComplianceClusters, complianceClustersFromDB.GetClusters(database.Pending))
eventCompliance.PendingComplianceClusters, allClustersOnDB)

batchLocalCompliances := []models.LocalStatusCompliance{}
batchLocalCompliances = append(batchLocalCompliances, compliantCompliances...)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -309,7 +309,7 @@ data:
"mode": "absolute",
"steps": [
{
"color": "yellow",
"color": "orange",
"value": null
}
]
Expand Down
Loading

0 comments on commit 112b353

Please sign in to comment.