Repository navigation
ci: lint and render every chart on every push, gated by ci_check_result - #452
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
🟡 Changes recommended
The workflow lacks a pull_request trigger, so fork PRs can remain blocked waiting for the required ci_check_result check when CI does not run in the fork.
1 open issue
What changed in this PR
Adds chart validation on every branch push and manual dispatch, preparing a required CI status check for the repository’s change-control requirements.
Changes:
- Calls the reusable Helm workflow at
v9to validate all five charts underparcellab. - Adds
ci_check_result, which fails when Helm validation fails or is cancelled. - Leaves release-time validation and publishing unchanged.
| File | Description |
|---|---|
.github/workflows/ci.yaml |
Adds push-triggered Helm validation and an aggregate status check for branch protection. |
Review effort: Balanced
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b8379da81d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Summary
Every PR has to run CI, and the result becomes a required status check. Until now, charts only lint/template/kubeval'd the charts on push to
main(inrelease.yaml), and PRs got path-filtered YAML/JSON lint at most.Description
New
ci.yaml, run on every push:helm: the new reusableparcelLab/cihelm.yamlwithchartsPath: parcellab. It runshelm dependency build+helm lint+helm template(skipped for thecommonlibrary chart) for all 5 charts.ci_check_result: fails ifhelmfailed or was cancelled. This becomes the required check.release.yamlis unchanged, including kubeval before publishing.Dependency
Uses
helm.yaml@v9from parcelLab/ci#158 (released in v9.2.2).