Skip to content

feat(helm): add extraContainers and extraVolumes support to deployment#2888

Open
rzala wants to merge 1 commit intoaquasecurity:mainfrom
rzala:feature/helm-extra-containers
Open

feat(helm): add extraContainers and extraVolumes support to deployment#2888
rzala wants to merge 1 commit intoaquasecurity:mainfrom
rzala:feature/helm-extra-containers

Conversation

@rzala
Copy link
Copy Markdown

@rzala rzala commented Feb 13, 2026

Summary

  • Adds extraContainers and extraVolumes values to inject sidecar containers and additional volumes into the trivy-operator deployment pod.
  • When alternateReportStorage is enabled, the report PVC is automatically mounted into each extra container so sidecars can access report files without manual volume configuration.
  • Refactors the volumes block to cleanly combine volumes, alternateReportStorage, and extraVolumes under a single conditional.

Test plan

  • helm template with defaults produces unchanged output
  • helm template with extraContainers and extraVolumes set renders sidecar and volume correctly
  • helm template with alternateReportStorage.enabled=true + extraContainers auto-mounts the PVC into sidecars
  • helm template with only alternateReportStorage.enabled=true (no extraContainers) still works as before

Add support for injecting arbitrary sidecar containers and extra
volumes into the trivy-operator Deployment via values.yaml.

When alternateReportStorage is enabled, the report PVC volume is
automatically mounted into each extra container, allowing sidecars
(log shippers, report exporters, monitoring agents) to access scan
report files without additional configuration.

New values:
- extraContainers: list of container specs to inject alongside the operator
- extraVolumes: list of volume specs to add to the pod
@rzala rzala marked this pull request as ready for review February 13, 2026 12:37
@rzala rzala requested review from afdesk and simar7 as code owners February 13, 2026 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant