Skip to content

chore(ci): drop informational Trivy fs scan (no image to migrate to grype) - #6

Merged
mattmattox merged 1 commit into
masterfrom
chore/trivy-to-grype
Jul 10, 2026
Merged

chore(ci): drop informational Trivy fs scan (no image to migrate to grype)#6
mattmattox merged 1 commit into
masterfrom
chore/trivy-to-grype

Conversation

@mattmattox

Copy link
Copy Markdown
Contributor

Summary

  • Removes the "Run Trivy vulnerability scanner" step from ci.yaml's security job.
  • This Trivy step was an informational, non-blocking filesystem/source scan (scan-type: fs, scan-ref: ., exit-code: 0) — it never gated CI and never scanned a container image.
  • This repo does not publish images to Harbor (harbor.support.tools); the build/release jobs push only to ghcr.io. The org-wide SupportTools/ci-runners/.github/workflows/grype-scan.yml@main reusable workflow scans container images, so there is nothing here for it to scan/replace.
  • Per the migration guidance, an image-less fs scan with no Harbor target is dropped rather than migrated to grype.
  • No .trivyignore existed, so no .grype.yaml was needed.
  • Gosec (SARIF, -no-fail, non-blocking) is untouched and remains the source-level security scan for this repo.

Test plan

  • python3 -c 'import yaml; yaml.safe_load(open(".github/workflows/ci.yaml"))' succeeds
  • No remaining trivy references anywhere under .github/workflows/
  • CI checks pass on this PR (lint/test/build/helm/security jobs unaffected by removal)

Trivy in ci.yaml scanned the source filesystem (scan-type: fs,
scan-ref: .) with exit-code 0, not a built image, and this repo does
not publish any image to Harbor (only ghcr.io). The org-wide grype-scan
reusable workflow only scans container images, so there is nothing for
it to replace here — the fs scan is dropped rather than migrated.
@mattmattox
mattmattox merged commit 156d5bf into master Jul 10, 2026
14 checks passed
@mattmattox
mattmattox deleted the chore/trivy-to-grype branch July 10, 2026 02:51
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.

1 participant