Skip to content

ci: add Trivy vulnerability scanning workflow - #242

Merged
btwshivam merged 1 commit into
optiqor:mainfrom
akandeadedayo01:ci/trivy-vulnerability-scan
Jul 19, 2026
Merged

ci: add Trivy vulnerability scanning workflow#242
btwshivam merged 1 commit into
optiqor:mainfrom
akandeadedayo01:ci/trivy-vulnerability-scan

Conversation

@akandeadedayo01

@akandeadedayo01 akandeadedayo01 commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

What

Adds a dedicated GitHub Actions security workflow that runs Trivy filesystem vulnerability scanning and
uploads SARIF results to GitHub Security.

Why

Fixes #234

How

The workflow runs on pushes to main, pull requests targeting main, and manual workflow_dispatch. It
uses aquasecurity/trivy-action to generate a SARIF report, then uploads it with github/codeql-action/ upload-sarif; forked PRs upload the SARIF as an artifact because they generally cannot write code scanning
results.

Testing

  • go build ./... passes
  • go test ./... passes
  • go vet ./... passes
  • golangci-lint run ./... passes
  • Tested locally with: git diff --check
  • N/A — CI workflow only
  • sudo ./bin/bpf-verify --read 5s confirms 6/6 programs still load
  • ./scripts/verify.sh passes (or specific phase: ./scripts/verify.sh quality)

Checklist

  • PR title follows Conventional Commits (feat(scope): subject)
  • All commits are DCO-signed (git commit -s)
  • No unrelated changes pulled in
  • Documentation updated where user-visible behavior changed
  • Added/updated tests for new code paths
  • If a new doctor rule, paired with a chaos scenario in scripts/verify.sh

Signed-off-by: Necst <akandeadedayo01@gmail.com>
@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown

🚀 First PR — welcome aboard!

A few things to expect:

  1. CI: every PR runs build + race tests + lint + (eventually) the kernel matrix. If something fails, the log will tell you exactly which gate.
  2. DCO: every commit needs Signed-off-by:git commit -s adds it automatically.
  3. Conventional Commits: PR titles like feat(doctor): add new rule or fix(bpf): handle X. We squash-merge by default.
  4. Review: a maintainer will review within 72 hours. Suggestions are conversations, not orders — push back if something doesn't fit your context.

If you get stuck, reply here or jump to Discussions. We want this PR to land.

@github-actions github-actions Bot added testing Tests and test coverage level:critical Touches BPF, security, or release surfaces (auto-applied) labels Jul 4, 2026
@akandeadedayo01

Copy link
Copy Markdown
Contributor Author

Fixes #234

@btwshivam
btwshivam merged commit 6d18992 into optiqor:main Jul 19, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level:critical Touches BPF, security, or release surfaces (auto-applied) testing Tests and test coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Add Trivy Vulnerability Scanning to GitHub Actions CI

2 participants