Skip to content

Add reports server chart installation to the kyverno chart

Placeholder Ammar Yasser requested to merge kyv-report-srv into main

Explanation This PR implements a unified Helm chart for Kyverno that optionally includes the Reports-Server as a subchart, The changes allow users to enable or disable Reports-Server via a configuration option, ensure Kyverno waits for Reports-Server to be ready before starting, and configure Kyverno to use Reports-Server for policy reports. The implementation maintains compatibility with future openreports integration by keeping its dependency conditional.

Related issue Closes #13739

Milestone of this PR /milestone 1.16.0

Documentation (required for features) My PR contains new or altered behavior to Kyverno installation .

What type of PR is this /kind feature

Proposed Changes The following changes were made to the Kyverno Helm chart in charts/kyverno:

Chart.yaml:

Added Reports-Server as a conditional dependency:

dependencies:

name: reports-server
version: "0.1.5"
repository: "https://kyverno.github.io/reports-server/"
condition: reportsServer.enabled
Version 0.1.5
values.yaml:

Added reportsServer configuration:

reportsServer:

   # -- Kyverno reports-server is used in your cluster
         enabled: false

   # -- Install CRDs required by reports-server
        installCRDs: true

Proof Manifests Testing Performed

Tests were conducted using a local Kind cluster (kind create cluster --name kyverno-test):

Linting:

helm lint charts/kyverno
Result: 1 chart(s) linted, 0 chart(s) failed.

Checklist [x] I have read the contributing guidelines. [x] I have read the PR documentation guide and followed the process including adding proof manifests to this PR. This is a bug fix and I have added unit tests that prove my fix is effective. This is a feature and I have added CLI tests that are applicable. My PR needs to be cherry picked to a specific release branch which is . [x] My PR contains new or altered behavior to Kyverno and CLI support should be added and my PR doesn't contain that functionality.

Merge request reports