Skip to content

feat: 👷🔒 Security scans workflow#63

Merged
evaline-ju merged 9 commits intokagenti:mainfrom
evaline-ju:security-scan
Mar 11, 2026
Merged

feat: 👷🔒 Security scans workflow#63
evaline-ju merged 9 commits intokagenti:mainfrom
evaline-ju:security-scan

Conversation

@evaline-ju
Copy link
Contributor

@evaline-ju evaline-ju commented Mar 10, 2026

Summary

  • Adds permissions and dependency pins for existing CI workflow for linting, unit tests
  • Adds security scan workflow, borrowed liberally from https://github.com/kagenti/kagenti/blob/main/.github/workflows/security-scans.yaml. Note gitleaks as commented in main kagenti would require a license
  • Start to address security scan issues - the readOnlyRootFilesystem for the ext-proc deployment was not quick to fix, so this is added to a .trivyignore ignore file for now
  • proto-build.sh script had to be updated to work with the "latest" envoy_data_plane library used to help build protos for this project, as it was not previously pinned. Team members say this was working with commit "bb0c0f1e912faa641cb5d9ccf1e2ef002945467f" (one commit after tag: 2.0.0b1 previously). The "latest" tag for which this is working is now incorporated as a "checkout" in the script

Related issue(s)

Addresses part of #55

Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
@github-advanced-security
Copy link

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
@evaline-ju evaline-ju changed the title feat: 👷 Security scan workflow feat: 👷🔒 Security scan workflow Mar 10, 2026
@evaline-ju evaline-ju changed the title feat: 👷🔒 Security scan workflow feat: 👷🔒 Security scans workflow Mar 10, 2026
Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
@evaline-ju evaline-ju marked this pull request as ready for review March 10, 2026 21:08
Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
@evaline-ju evaline-ju merged commit f7ec9a8 into kagenti:main Mar 11, 2026
12 checks passed
@evaline-ju evaline-ju deleted the security-scan branch March 11, 2026 13:40
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.

2 participants