fix resource mapping reference issue

Signed-off-by: Frank Jogeleit <[email protected]>
kyverno · Nov 27, 2023 · a039531 · a039531
1 parent 4d18403
commit a039531
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,12 +1,14 @@
 # Changelog
 
 # Next
+* Policy Reporter v2.17.4
+    * Fix Result Resource mapping
 * Policy Reporter v2.17.3
     * Fix handling if metric cache fallback
 
 # 2.21.2
 * Policy Reporter v2.17.2
-    * Fix ID generation for Policy Reports which using `scope` as resource reference
+    * Fix ID generation for PolicyReportResults which using `scope` as resource reference
     * Add optional `message` metric label
 * Helm Chart
     * fix: Add chart parameters for setting `revisionHistoryLimit` [[#363](https://github.com/kyverno/policy-reporter/pull/363) by [bodgit](https://github.com/bodgit)]

diff --git a/pkg/kubernetes/queue.go b/pkg/kubernetes/queue.go
@@ -115,19 +115,22 @@ func (q *Queue) processNextItem() bool {
 
 	q.handleErr(err, key)
 
-	q.debouncer.Add(report.LifecycleEvent{Type: event, PolicyReport: polr})
+	q.debouncer.Add(report.LifecycleEvent{Type: event, PolicyReport: mapResource(polr)})
 
 	return true
 }
 
 // each result needs to know its resource it belongs to, to generate internal unique IDs
 func mapResource(polr pr.ReportInterface) pr.ReportInterface {
-	for _, r := range polr.GetResults() {
+	results := polr.GetResults()
+	for i, r := range results {
 		scope := polr.GetScope()
 
 		if len(r.Resources) == 0 && scope != nil {
 			r.Resources = append(r.Resources, *scope)
 		}
+
+		results[i] = r
 	}
 
 	return polr