Skip to content

Add trivy scan IaC#2000

Open
ggjulio wants to merge 1 commit intoopen-telemetry:mainfrom
ggjulio:trivy-scan
Open

Add trivy scan IaC#2000
ggjulio wants to merge 1 commit intoopen-telemetry:mainfrom
ggjulio:trivy-scan

Conversation

@ggjulio
Copy link
Copy Markdown

@ggjulio ggjulio commented Dec 22, 2025

Trivy reports +4000 issues in the security tabs. (https://github.com/ggjulio/opentelemetry-helm-charts/security/code-scanning)
But the scan can still be merged without breaking anything, later on issues can either be fixed, ignored (.trivyconfig) or closed in the security tab.

image image image

@ggjulio ggjulio requested a review from a team as a code owner December 22, 2025 10:11
Comment thread .devcontainer/Dockerfile Outdated
uses: actions/checkout@v4

- name: Run Trivy vulnerability scanner in IaC mode
uses: aquasecurity/trivy-action@0.33.1
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is getting scanned by trivy in the new action? We don't have any code in this repository, only references to docker images.

Copy link
Copy Markdown
Author

@ggjulio ggjulio Dec 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trivy don't scan for code, but scan for "misconfig" in IaC files. (In this case, helm charts)

While scanning third parties charts we, we found ~18 flagged misconfigs in the otel chart. (Didn't looked if they were legit though, but all other third parties we use have none.)

Useful links:

@github-actions
Copy link
Copy Markdown
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Jan 15, 2026
@ggjulio ggjulio changed the title Add trivy scan IaC + devcontainer definition. Add trivy scan IaC Jan 15, 2026
@github-actions github-actions Bot removed the Stale label Jan 16, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Jan 30, 2026
@ggjulio
Copy link
Copy Markdown
Author

ggjulio commented Jan 31, 2026

@TylerHelmuth , i'll close the pr ?

@github-actions github-actions Bot removed the Stale label Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[security] audit repository tooling

2 participants