Skip to content

Update default Helm version to 4.2.0 in CI/CD workflows#96

Merged
omarghatasheh merged 2 commits into
mainfrom
helmUpdate
Jun 10, 2026
Merged

Update default Helm version to 4.2.0 in CI/CD workflows#96
omarghatasheh merged 2 commits into
mainfrom
helmUpdate

Conversation

@MusaMisto

@MusaMisto MusaMisto commented Jun 10, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Chores
    • Updated CI/CD deployment tooling to the latest compatible version for improved reliability and security.

@MusaMisto MusaMisto self-assigned this Jun 10, 2026
@MusaMisto MusaMisto added the enhancement New feature or request label Jun 10, 2026
@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@MusaMisto, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 43 minutes and 33 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2939f102-dd65-474e-a5ee-475ff123fcc4

📥 Commits

Reviewing files that changed from the base of the PR and between a3f8a75 and 67f9e29.

📒 Files selected for processing (5)
  • .github/actions/helm-generic/action.yml
  • .github/workflows/ci-helm.yaml
  • .github/workflows/helm-deploy-values.yml
  • AGENTS.md
  • README.md
📝 Walkthrough

Walkthrough

This PR updates the default Helm version in a reusable GitHub Actions workflow from version 3.2.4 to 4.2.0. The change applies to both the workflow_call and workflow_dispatch input declarations in the same workflow file.

Changes

Helm Version Update

Layer / File(s) Summary
Helm version defaults
.github/workflows/generic-gateway-chart-cicd.yml
Both workflow_call and workflow_dispatch input defaults for helm-version are updated from 3.2.4 to 4.2.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • simplify9/.github#56: Directly related as this PR updates the helm-version default in the same workflow file that was added in that PR.

Suggested reviewers

  • omarghatasheh

Poem

🐰 A helm update hops in the night,
From 3.2.4 to 4.2.0, shining bright,
Workflow calls and dispatches align,
Smooth sailing with versions so fine!
thump thump 🚀

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: updating the default Helm version to 4.2.0 in CI/CD workflows.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch helmUpdate

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/generic-gateway-chart-cicd.yml:
- Around line 19-20: The workflow sets helm-version default to "4.2.0" (used by
azure/setup-helm@v5) which may be incompatible with existing Helm v3
expectations; change the default for helm-version in the
generic-gateway-chart-cicd workflow to the repo's agreed Helm 3 pin (e.g., match
'helm-version' used in sw-cicd.yml or README, such as '3.21.0' or '3.2.4'), or
remove the hard default and require callers to pass a version, and add a short
comment near the helm-version declaration describing the supported Helm major
version to prevent accidental upgrades; ensure azure/setup-helm@v5 usage stays
compatible with the chosen helm-version.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 274ed91b-e997-47ae-944b-ef8516e4f03a

📥 Commits

Reviewing files that changed from the base of the PR and between d0d9c6a and a3f8a75.

📒 Files selected for processing (1)
  • .github/workflows/generic-gateway-chart-cicd.yml

Comment on lines +19 to 20
default: "4.2.0"
type: string

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Read-only verification: find Helm version usage/docs drift in repo
rg -n -C2 'helm-version:\s*' .github/workflows .github/actions README.md

Repository: simplify9/.github

Length of output: 2720


🌐 Web query:

Is Helm CLI version 4.2.0 officially released and what are the documented breaking changes vs Helm 3 that impact helm lint, helm template, or helm package?

💡 Result:

Yes, Helm CLI version 4.2.0 is officially released [1][2]. It was published on May 14, 2026 [1][3]. Helm 4 introduces several architectural and breaking changes compared to Helm 3. While many workflows remain compatible, specific changes impact how you use commands like helm lint, helm template, and helm package [4][5]. Key breaking changes and impacts include: 1. Linting (helm lint) - The lint package has been refactored [6]. Specifically, it was moved to be part of each chart version to improve validation, including enhanced support for CRD directories [6][7][8]. - The linting process now returns early if the templates/ directory does not exist [6]. - Legacy Helm v2 template lint rules have been removed [6]. 2. Templating (helm template) - Post-renderers, which are used during rendering, are now implemented as plugins [5][8]. Consequently, you can no longer pass an executable directly to the post-renderer flag; you must use a configured plugin name instead [5]. - Several flags (e.g., --hide-notes and --render-subchart-notes) are deprecated and have no effect, as template output no longer includes notes by default [5]. - New template functions such as mustToYaml and mustToJson have been added [6][7][9]. 3. Packaging (helm package) - The internal package structure has been moved to versioned packages (e.g., pkg/chart/v2) to support future chart API versions [5][7][8]. - Plugin packaging, signing, and verification processes were overhauled, including the removal of unnecessary file I/O operations [6][8]. General breaking changes also include: - Removal of the HELM_EXPERIMENTAL_OCI environment variable; OCI registry support is now enabled by default [10]. - Flag renaming for certain commands (e.g., --atomic changed to --rollback-on-failure and --force to --force-replace) [7]. - Plugin system overhaul based on WebAssembly (WASM), which may require existing plugins to be updated for compatibility [7][10]. - If you use Helm as a Go library, the module path has changed from helm.sh/helm/v3 to helm.sh/helm/v4 [10][8]. For detailed migration steps and a full list of changes, refer to the official Helm release notes and documentation [11][4][5].

Citations:


🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Show the claimed default areas in the workflow
echo "== generic-gateway-chart-cicd.yml (around line 19) =="
sed -n '1,120p' .github/workflows/generic-gateway-chart-cicd.yml | cat -n

echo
echo "== generic-gateway-chart-cicd.yml (around line 70-110) =="
sed -n '70,120p' .github/workflows/generic-gateway-chart-cicd.yml | cat -n

echo
echo "== helm-package-push/action.yml (around helm-version handling + setup-helm) =="
sed -n '1,140p' .github/actions/helm-package-push/action.yml | cat -n

echo
echo "== helm-deploy/action.yml (helm-version handling, if relevant) =="
sed -n '1,140p' .github/actions/helm-deploy/action.yml | cat -n

Repository: simplify9/.github

Length of output: 18565


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== generic-gateway-chart-cicd.yml: where inputs.helm-version is forwarded =="
rg -n "helm-version|inputs\.helm-version" .github/workflows/generic-gateway-chart-cicd.yml

echo
echo "== README.md: Helm version mentions =="
rg -n "Helm|helm-version|v3\.|Helm 3|Helm 4" README.md .github/workflows .github/actions || true

Repository: simplify9/.github

Length of output: 23860


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== generic-gateway-chart-cicd.yml: around first Set up Helm + Helm lint =="
sed -n '160,240p' .github/workflows/generic-gateway-chart-cicd.yml | cat -n

echo
echo "== generic-gateway-chart-cicd.yml: around packaging/push Helm setup (second Set up Helm) =="
sed -n '280,360p' .github/workflows/generic-gateway-chart-cicd.yml | cat -n

echo
echo "== generic-gateway-chart-cicd.yml: find helm commands/flags =="
rg -n "helm (lint|template|package)|--post-renderer|post-renderer|hide-notes|render-subchart-notes|apiVersion|HELM_EXPERIMENTAL_OCI" .github/workflows/generic-gateway-chart-cicd.yml

Repository: simplify9/.github

Length of output: 8463


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== generic-gateway-chart-cicd.yml: around first Set up Helm + Helm lint =="
sed -n '160,240p' .github/workflows/generic-gateway-chart-cicd.yml | cat -n

echo
echo "== generic-gateway-chart-cicd.yml: around packaging/push Helm setup (second Set up Helm) =="
sed -n '280,360p' .github/workflows/generic-gateway-chart-cicd.yml | cat -n

echo
echo "== generic-gateway-chart-cicd.yml: find helm commands/flags =="
rg -n "helm (lint|template|package)|--post-renderer|post-renderer|hide-notes|render-subchart-notes|apiVersion|HELM_EXPERIMENTAL_OCI" .github/workflows/generic-gateway-chart-cicd.yml

Repository: simplify9/.github

Length of output: 8463


🏁 Script executed:

#!/bin/bash
# Read-only: find Helm version usage/docs drift and where generic-gateway-chart-cicd uses inputs.helm-version
rg -n -C2 'helm-version:\s*|version:\s*\$\{\{\s*inputs\.helm-version\s*\}\}|helm (lint|template|package)\b' .github/workflows .github/actions README.md

Repository: simplify9/.github

Length of output: 9996


🏁 Script executed:

#!/bin/bash
set -euo pipefail
rg -n "helm-package-push|helm-deploy|helm-generic|actions/helm-" .github/workflows/generic-gateway-chart-cicd.yml

Repository: simplify9/.github

Length of output: 43


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== Search for composite action usage in generic-gateway-chart-cicd.yml =="
rg -n "uses:\s*\.github/actions/|uses:\s*.*helm-" .github/workflows/generic-gateway-chart-cicd.yml

echo
echo "== Search for 'helm package' / 'helm template' blocks =="
rg -n "helm (lint|template|package)\b" .github/workflows/generic-gateway-chart-cicd.yml -n

echo
echo "== Show exact 'Set up Helm' steps wiring inputs.helm-version =="
rg -n "Set up Helm|azure/setup-helm@v5|version:\s*\$\{\{\s*inputs\.helm-version\s*\}\}" .github/workflows/generic-gateway-chart-cicd.yml -n

echo
echo "== Confirm whether generic-gateway-chart-cicd.yml calls helm-package-push action =="
rg -n "helm-package-push" .github/workflows/generic-gateway-chart-cicd.yml || true

Repository: simplify9/.github

Length of output: 135


🏁 Script executed:

#!/bin/bash
# Read-only: find Helm version usage/docs drift and where this workflow uses inputs.helm-version
rg -n -C2 'helm-version:\s*"4\.2\.0"|azure/setup-helm@v5|helm (lint|template|package)\b' .github/workflows/generic-gateway-chart-cicd.yml .github/workflows/sw-cicd.yml README.md

Repository: simplify9/.github

Length of output: 5020


🏁 Script executed:

#!/bin/bash
set -euo pipefail
rg -n 'default:\s*"4\.2\.0"|azure/setup-helm@v5|helm (lint|template|package)\b' .github/workflows/generic-gateway-chart-cicd.yml
rg -n "helm-version:\s*'3\.2\.4'|v3\.21\.0" .github/workflows/sw-cicd.yml README.md

Repository: simplify9/.github

Length of output: 1079


Verify Helm 4.2.0 compatibility before defaulting helm-version in generic-gateway-chart-cicd

File: .github/workflows/generic-gateway-chart-cicd.yml (Lines 19-20, also 85-86)

default: "4.2.0"
type: string

The workflow now defaults helm-version to 4.2.0 (for both workflow_call and workflow_dispatch), installs it via azure/setup-helm@v5, and uses it for helm lint, helm template, and helm package. The repo still documents/pins Helm 3 expectations (.github/workflows/sw-cicd.yml uses helm-version: '3.2.4', and README.md lists helm-version as v3.21.0), so Helm 4’s breaking behavioral changes could affect lint/template/package outputs across callers.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.github/workflows/generic-gateway-chart-cicd.yml around lines 19 - 20, The
workflow sets helm-version default to "4.2.0" (used by azure/setup-helm@v5)
which may be incompatible with existing Helm v3 expectations; change the default
for helm-version in the generic-gateway-chart-cicd workflow to the repo's agreed
Helm 3 pin (e.g., match 'helm-version' used in sw-cicd.yml or README, such as
'3.21.0' or '3.2.4'), or remove the hard default and require callers to pass a
version, and add a short comment near the helm-version declaration describing
the supported Helm major version to prevent accidental upgrades; ensure
azure/setup-helm@v5 usage stays compatible with the chosen helm-version.

@omarghatasheh omarghatasheh merged commit 28b81f9 into main Jun 10, 2026
2 checks passed
@omarghatasheh omarghatasheh deleted the helmUpdate branch June 10, 2026 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants