Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

helm - trivy-server should use trivy.repository:trivy.tag for image if defined #729

Closed
jrhunger opened this issue Nov 23, 2022 · 1 comment · Fixed by #732
Closed

helm - trivy-server should use trivy.repository:trivy.tag for image if defined #729

jrhunger opened this issue Nov 23, 2022 · 1 comment · Fixed by #732
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@jrhunger
Copy link
Contributor

What steps did you take and what happened:

Deployed v0.7.0 using with built-in server enabled, in a cluster that has a proxy firewall (with internal-signed-cert) between it and ghcr.io. Top-level values.yaml specifies trivy.repository and trivy.tag to point at our custom trivy image that includes CA cert trust for our internal certificate.

trivy-server-0 pod sent to Crash Loop Backoff due to cert mismatch when downloading DB.

What did you expect to happen:

I expected that trivy-server would use the defined trivy image which trusts our internal cert and would successfully download the DB and continue running.

Anything else you would like to add:
Should be fixable by replacing the hardcoded image line in templates/trivy-server.yaml with this one:

image: {{ .Values.trivy.repository }}:{{ .Values.trivy.tag | default .Chart.AppVersion }}

I tested this with local modification to helm chart and it works.

Environment:

  • Trivy-Operator version (use trivy-operator version): v0.7.0
  • Kubernetes version (use kubectl version): 1.24.3
  • OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): Oracle Linux 7
@jrhunger jrhunger added the kind/bug Categorizes issue or PR as related to a bug. label Nov 23, 2022
@adrianmarcu18
Copy link

I would also like this to be fixed. To be mentioned that there are some other hardcoded values in the template, such as resources.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants