chore(chart-deps): update policy-reporter to version 3.9.1 - #3461
chore(chart-deps): update policy-reporter to version 3.9.1#3461svcAPLBot wants to merge 20 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR bumps the vendored policy-reporter Helm chart to 3.9.1 and aligns chart defaults/templates/README with the updated upstream chart behavior.
Changes:
- Update chart metadata (Chart.yaml, chart-index) and various default image tags/values to match policy-reporter chart v3.9.1.
- Add new configuration surfaces (e.g., Microsoft Graph API email reports, autoMemoryLimit) and wire them into rendered config templates.
- Adjust rendered resources (HTTPRoute backendRefs, ServiceMonitor gating, CronJob env vars, RBAC additions) to reflect upstream chart updates.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| charts/policy-reporter/values.yaml | Updates defaults (image tags, new graphAPI + autoMemoryLimit, ServiceMonitor flag, source filter defaults). |
| charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml | Updates NamespaceCustomBoard CRD schema and adds templated annotations support. |
| charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_customboards.yaml | Updates CustomBoard CRD schema and documentation strings. |
| charts/policy-reporter/templates/ui/httproute.yaml | Makes backendRefs explicit (group/kind/name) for Gateway API. |
| charts/policy-reporter/templates/plugins/kyverno/httproute.yaml | Same backendRefs explicitness for kyverno plugin route. |
| charts/policy-reporter/templates/monitoring/servicemonitor.yaml | Gates ServiceMonitor rendering on new monitoring.serviceMonitor.enabled. |
| charts/policy-reporter/templates/httproute.yaml | Makes backendRefs explicit (group/kind/name) for core route. |
| charts/policy-reporter/templates/cronjob-violations-report.yaml | Adds POD_NAMESPACE env var and ensures env: always exists. |
| charts/policy-reporter/templates/cronjob-summary-report.yaml | Adds POD_NAMESPACE env var and ensures env: always exists. |
| charts/policy-reporter/templates/clusterrole.yaml | Adds apps/replicasets read permission. |
| charts/policy-reporter/README.md | Updates version/appVersion badges and values documentation tables. |
| charts/policy-reporter/configs/email-reports.tmpl | Wires emailReports.graphAPI into generated config. |
| charts/policy-reporter/configs/core.tmpl | Wires autoMemoryLimit into generated config. |
| charts/policy-reporter/Chart.yaml | Bumps chart version to 3.9.1 and appVersion to 3.9.0 per upstream chart. |
| chart/chart-index/Chart.yaml | Updates chart dependency version reference to 3.9.1. |
| annotations: | ||
| controller-gen.kubebuilder.io/version: v2.4.5-0.20250927090338-8022a0e22018+incompatible.dirty | ||
| {{- if .Values.annotations }} | ||
| annotations: | ||
| {{- toYaml .Values.annotations | nindent 4 }} | ||
| {{- end }} |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (2)
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml:9
metadatacurrently contains twoannotations:keys: an emptyannotations:line plus a conditionalannotations:block. This renders invalid/ambiguous YAML (duplicate keys) and can cause CRD apply failures or unexpected annotation behavior.
annotations:
{{- if .Values.annotations }}
annotations:
{{- toYaml .Values.annotations | nindent 4 }}
{{- end }}
charts/policy-reporter/templates/monitoring/servicemonitor.yaml:1
- The ServiceMonitor gate now requires
.Values.monitoring.serviceMonitor.enabledto be set. Onhelm upgrade --reuse-values, newly-added defaults are not merged, so this can evaluate to false and unexpectedly stop rendering the ServiceMonitor even thoughmonitoring.enabledis true. Consider defaulting this flag totrueat template time to preserve prior behavior for existing installs.
{{- if and .Values.monitoring.enabled .Values.monitoring.serviceMonitor.enabled }}
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (2)
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml:9
- The CRD template renders an empty
metadata.annotations:line and can also render a secondannotations:key when.Values.annotationsis set, producing invalid YAML (duplicate key / empty value). This will breakhelm template/applyfor the UI CRDs.
annotations:
{{- if .Values.annotations }}
annotations:
{{- toYaml .Values.annotations | nindent 4 }}
{{- end }}
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_customboards.yaml:184
renderOptionsis not a namespace selector, but its schema description currently says "NamespaceSelector allows to select visualized namespaces", which is misleading for CRD users and generated docs.
renderOptions:
description: NamespaceSelector allows to select visualized namespaces
properties:
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.
Suppressed comments (2)
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml:9
metadatacurrently renders an emptyannotations:key and then (optionally) renders a secondannotations:key. This results in duplicate YAML keys and invalid output when.Values.annotationsis set (and can also produce a danglingannotations:when it is not). Remove the strayannotations:line so only a single annotations map is rendered undermetadata.
annotations:
{{- if .Values.annotations }}
annotations:
{{- toYaml .Values.annotations | nindent 4 }}
{{- end }}
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_customboards.yaml:185
- The
renderOptionsschema description looks copy/pasted (it currently says it's a NamespaceSelector). This is misleading in generated CRD docs andkubectl explain. Update the description to reflect whatrenderOptionsactually configures.
renderOptions:
description: NamespaceSelector allows to select visualized namespaces
properties:
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.
Suppressed comments (1)
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml:10
- In
ui.policyreporter.kyverno.io_namespacecustomboards.yaml,metadatacurrently contains an unconditionalannotations:key (with no value) followed by a conditional secondannotations:block. This can render invalid YAML / invalid CRD metadata (duplicate key, orannotations: nullwhen.Values.annotationsis empty). Remove the strayannotations:line and keep only the conditional block.
metadata:
annotations:
{{- if .Values.annotations }}
annotations:
{{- toYaml .Values.annotations | nindent 4 }}
{{- end }}
name: namespacecustomboards.ui.policyreporter.kyverno.io
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.
Suppressed comments (1)
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml:9
- The CRD template currently renders an empty
metadata.annotations:key and then conditionally renders a secondannotations:key when.Values.annotationsis set. This results in duplicate YAML keys undermetadata, which can break YAML parsing / CRD application.
annotations:
{{- if .Values.annotations }}
annotations:
{{- toYaml .Values.annotations | nindent 4 }}
{{- end }}
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.
Suppressed comments (1)
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml:9
- This template now renders an empty
metadata.annotations:key unconditionally and (when.Values.annotationsis set) renders a secondannotations:key, producing invalid YAML for the CRD metadata. Kubernetes expectsmetadata.annotationsto be a map (not null), and duplicate keys can fail manifest parsing/apply.
annotations:
{{- if .Values.annotations }}
annotations:
{{- toYaml .Values.annotations | nindent 4 }}
{{- end }}
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.
Suppressed comments (2)
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml:8
metadatacurrently renders twoannotationskeys (one empty and one conditional), which produces invalid YAML (duplicate key) when.Values.annotationsis set, and a meaninglessannotations: nullwhen it is not. Remove the unconditionalannotations:line and keep a single conditional annotations block.
metadata:
annotations:
{{- if .Values.annotations }}
annotations:
{{- toYaml .Values.annotations | nindent 4 }}
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_customboards.yaml:184
- The
renderOptionsschema description is a copy/paste fromnamespaces("NamespaceSelector...") and is misleading for CRD consumers. Update it to describe whatrenderOptionsactually controls.
description: NamespaceSelector allows to select visualized namespaces
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.
Suppressed comments (1)
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml:10
metadatacontains twoannotations:keys: an unconditional emptyannotations:line followed by a conditionalannotations:block. This renders invalid YAML (duplicate key) and can cause Helm/Kubernetes to reject the CRD manifest.
metadata:
annotations:
{{- if .Values.annotations }}
annotations:
{{- toYaml .Values.annotations | nindent 4 }}
{{- end }}
name: namespacecustomboards.ui.policyreporter.kyverno.io
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.
Suppressed comments (1)
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml:9
metadatacurrently emits an emptyannotations:key and then conditionally emits a secondannotations:block, resulting in duplicate keys / invalid YAML (andmetadata.annotationsbecoming null when.Values.annotationsis unset). This CRD will fail to apply.
annotations:
{{- if .Values.annotations }}
annotations:
{{- toYaml .Values.annotations | nindent 4 }}
{{- end }}
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.
Suppressed comments (1)
charts/policy-reporter/templates/ui/ui.policyreporter.kyverno.io_namespacecustomboards.yaml:9
- The CRD template renders an
annotations:key unconditionally (line 5) and then conditionally renders a secondannotations:block. This produces duplicate YAML keys and can also renderannotations: nullwhen.Values.annotationsis unset, which is invalid for Kubernetes object metadata.
annotations:
{{- if .Values.annotations }}
annotations:
{{- toYaml .Values.annotations | nindent 4 }}
{{- end }}
This PR updates the dependency policy-reporter to version 3.9.1.