Skip to content

[chore] pin to helm v4 in workflows#2399

Open
jinja2 wants to merge 1 commit intomainfrom
fix-helm-vers
Open

[chore] pin to helm v4 in workflows#2399
jinja2 wants to merge 1 commit intomainfrom
fix-helm-vers

Conversation

@jinja2
Copy link
Copy Markdown
Collaborator

@jinja2 jinja2 commented Apr 25, 2026

Description:

Pin Helm to v4.1.4 in update_docker_images.yaml and release_drafter.yaml. Both workflows commit rendered manifests via make render (-> helm template) but they were silently falling back to the runner's pre-installed Helm 3.
The result is a render-flap loop: update_docker_images / release_drafter open a PR with manifests rendered by Helm 3 (example). Next time anything Helm-4-pinned runs make render, it reverts/rewrites those manifests.

Copilot AI review requested due to automatic review settings April 25, 2026 03:22
@jinja2 jinja2 requested review from a team as code owners April 25, 2026 03:22
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR pins Helm to v4.1.4 in the update_docker_images and release_drafter GitHub Actions workflows to prevent them from falling back to the runner’s preinstalled Helm 3, which was causing rendered-manifest “flap” PR loops.

Changes:

  • Add an explicit azure/setup-helm step to update_docker_images.yaml with version: v4.1.4.
  • Add an explicit azure/setup-helm step to release_drafter.yaml with version: v4.1.4.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/update_docker_images.yaml Pins Helm to v4.1.4 so make render / helm template uses the intended Helm version.
.github/workflows/release_drafter.yaml Pins Helm to v4.1.4 so release draft PRs render manifests consistently.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants