Skip to content

Add Sentinel CI workflow for workflow security scanning#15

Open
jpr5 wants to merge 1 commit into
mainfrom
sentinel-rollout/add-workflow
Open

Add Sentinel CI workflow for workflow security scanning#15
jpr5 wants to merge 1 commit into
mainfrom
sentinel-rollout/add-workflow

Conversation

@jpr5
Copy link
Copy Markdown
Contributor

@jpr5 jpr5 commented May 22, 2026

Summary

  • Adds .github/workflows/sentinel.yml — runs Sentinel on every PR and push to main
  • Configured in warn-only mode (fail-on-findings: false) so it surfaces findings without blocking merges
  • Part of org-wide Sentinel rollout (spec)

Details

Setting Value
Trigger pull_request + push to main
Severity threshold high
Blocking No (fail-on-findings: false)
Timeout 10 minutes
Permissions contents: read (least privilege)

Test plan

  • Verify workflow appears in Actions tab after merge
  • Confirm scan runs on next PR and produces annotations without failing the check

@jpr5 jpr5 force-pushed the sentinel-rollout/add-workflow branch from edb0f3d to 3e2cc53 Compare May 22, 2026 20:49
@jpr5
Copy link
Copy Markdown
Contributor Author

jpr5 commented May 22, 2026

Force-pushed with SHA-pinned actions to satisfy zizmor's unpinned-uses rule. Per the org-wide sentinel rollout, the floating-tag-for-easy-updates model is being replaced with SHA-pin + dependabot. Spec updated. Rest of the PR unchanged.

Part of org-wide sentinel rollout. Warn-only mode (fail-on-findings: false).
Spec: https://www.notion.so/copilotkit/3683aa381852818bacd8e14eb7233c22
@jpr5
Copy link
Copy Markdown
Contributor Author

jpr5 commented May 22, 2026

Promoted to blocking mode — this repo had zero sentinel findings on the warn-only scan, so we're skipping the 7-day warn-only observation period. fail-on-findings is now true; future PRs that introduce critical/high findings will block merge.

@jpr5 jpr5 force-pushed the sentinel-rollout/add-workflow branch from 3e2cc53 to 68f1c14 Compare May 22, 2026 21:32
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