Skip to content

fix: correct output for uncompressed logs#2652

Merged
simar7 merged 5 commits intoaquasecurity:mainfrom
afdesk:fix/suppres-logs
Jul 16, 2025
Merged

fix: correct output for uncompressed logs#2652
simar7 merged 5 commits intoaquasecurity:mainfrom
afdesk:fix/suppres-logs

Conversation

@afdesk
Copy link
Copy Markdown
Contributor

@afdesk afdesk commented Jul 11, 2025

Description

This PR keeps logs for scans with flag scanJobCompressLogs=false inside files instead of stdout.

Reproduction steps

$ helm install trivy-operator aquasecurity/trivy-operator \
     --create-namespace --namespace trivy-system \
     --set trivyOperator.scanJobCompressLogs=false

Testing

$ helm install trivy-operator aquasecurity/trivy-operator \
     --create-namespace --namespace trivy-system \
     --set trivyOperator.scanJobCompressLogs=false \
      --set image.tag=uncompressed --set image.repository=afdesk/trivy-operator \
      --set operator.scanJobTTL=30m --set trivy.resources.limits.memory=1Gi

Related issues

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

@github-actions github-actions bot added the bug label Jul 11, 2025
@afdesk afdesk marked this pull request as ready for review July 14, 2025 09:07
@afdesk afdesk requested a review from simar7 as a code owner July 14, 2025 09:07
@afdesk
Copy link
Copy Markdown
Contributor Author

afdesk commented Jul 14, 2025

@simar7 could you please take a look when you have time?

@simar7 simar7 merged commit 1725f9a into aquasecurity:main Jul 16, 2025
12 checks passed
@afdesk afdesk deleted the fix/suppres-logs branch August 6, 2025 04:30
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.

2 participants