ci: add helm chart ext-blockfrost-operator publish script#367
Conversation
Signed-off-by: Ales Verbic <verbotenj@blinklabs.io>
📝 WalkthroughWalkthroughThis pull request introduces a new GitHub Actions workflow file that automates publishing of the Blockfrost Operator Helm chart. The workflow triggers on pushes to the main branch when files in the 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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.
🧹 Nitpick comments (1)
.github/workflows/publish-ext-blockfrost-operator-helm-chart.yml (1)
20-21: Pin Helm CLI version for reproducible CI behavior.On lines 20–21,
azure/setup-helmis SHA-pinned but Helm itself defaults tolatest. Add an explicit version parameter to ensure consistent behavior across CI runs and align with other workflows in this repository (e.g.,publish-bind9-helm-chart.yml).Suggested patch
- name: Install Helm uses: azure/setup-helm@dda3372f752e03dde6b3237bc9431cdc2f7a02a2 # v5.0.0 + with: + version: v3.18.3🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/workflows/publish-ext-blockfrost-operator-helm-chart.yml around lines 20 - 21, The Helm setup step is SHA-pinned but doesn't fix the Helm CLI version; update the "Install Helm" step that uses azure/setup-helm@dda3372f752e03dde6b3237bc9431cdc2f7a02a2 to include a with: version: "<pin-a-specific-helm-version>" entry (e.g., version: "3.11.3") so the workflow installs a reproducible Helm CLI version instead of defaulting to latest.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.github/workflows/publish-ext-blockfrost-operator-helm-chart.yml:
- Around line 20-21: The Helm setup step is SHA-pinned but doesn't fix the Helm
CLI version; update the "Install Helm" step that uses
azure/setup-helm@dda3372f752e03dde6b3237bc9431cdc2f7a02a2 to include a with:
version: "<pin-a-specific-helm-version>" entry (e.g., version: "3.11.3") so the
workflow installs a reproducible Helm CLI version instead of defaulting to
latest.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: ef7f2287-4468-42cd-803d-4f2d7569c555
📒 Files selected for processing (1)
.github/workflows/publish-ext-blockfrost-operator-helm-chart.yml
Summary by cubic
Add a GitHub Actions workflow to package and publish the
ext-blockfrost-operatorHelm chart toghcr.ioon pushes tomain. This automates chart releases underghcr.io/<owner>/<repo>/charts.charts/ext-blockfrost-operatorand pushes toghcr.io/<owner>/<repo>/chartsvia Helm OCI.actions/checkoutandazure/setup-helmwithGITHUB_TOKENauth.Written for commit fc8f1ef. Summary will update on new commits.
Summary by CodeRabbit