chore(governance): add CI/CD governance baseline#14
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (7)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4bd2b56ad5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| jobs: | ||
| gates: | ||
| uses: ./.github/workflows/reusable-governance-gates.yml |
There was a problem hiding this comment.
Reference an existing reusable governance workflow
This job points to ./.github/workflows/reusable-governance-gates.yml, but that file is not present in this repository (repo-wide filename search returns no match), so the Governance Gates workflow cannot run when triggered on PRs/pushes. As written, the governance gate is effectively broken and won't execute its intended checks.
Useful? React with 👍 / 👎.
| shell: bash | ||
| run: | | ||
| set -euo pipefail | ||
| bash scripts/check-chitty-onboarding.sh .chittyconnect.yml |
There was a problem hiding this comment.
Call an onboarding check script that exists in the repo
The workflow invokes scripts/check-chitty-onboarding.sh, but there is no such script in this repository, so this step will fail with a command-not-found/file-not-found error on every PR and push to main. That makes the onboarding gate permanently red instead of validating configuration.
Useful? React with 👍 / 👎.
| run: | | ||
| set -euo pipefail | ||
| mkdir -p reports/secret-rotation | ||
| if bash scripts/onepassword-rotation-audit.sh .github/secret-catalog.json reports/secret-rotation; then |
There was a problem hiding this comment.
Point rotation audit to a real script path
This step executes scripts/onepassword-rotation-audit.sh, but that script does not exist in the repository, so the scheduled/manual audit cannot perform any rotation checks and will always hit the failure branch. The workflow currently cannot produce meaningful audit results until this path is corrected or the script is added.
Useful? React with 👍 / 👎.
Automated governance baseline remediation from org control loop.