Skip to content

workflows: Add retry logic for helm chart PR merge#43585

Merged
aanm merged 1 commit intomainfrom
pr/fix-release-helm
Jan 7, 2026
Merged

workflows: Add retry logic for helm chart PR merge#43585
aanm merged 1 commit intomainfrom
pr/fix-release-helm

Conversation

@aanm
Copy link
Copy Markdown
Member

@aanm aanm commented Jan 6, 2026

Add retry mechanism to handle concurrent patch releases when merging helm chart PRs. When a merge fails (e.g., due to conflicts from simultaneous patch releases), the workflow will re-run the release tool to update the PR with the latest changes and re-try 3 more times.

The retry logic ensures that transient merge failures due to race conditions between concurrent releases are automatically resolved without manual intervention.

Add retry mechanism to handle concurrent patch releases when merging
helm chart PRs. When a merge fails (e.g., due to conflicts from
simultaneous patch releases), the workflow will re-run the release tool
to update the PR with the latest changes and re-try 3 more times.

The retry logic ensures that transient merge failures due to race
conditions between concurrent releases are automatically resolved
without manual intervention.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm requested a review from joestringer January 6, 2026 16:17
@aanm aanm requested review from a team as code owners January 6, 2026 16:17
@aanm aanm requested a review from Artyop January 6, 2026 16:17
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jan 6, 2026
@aanm aanm enabled auto-merge January 6, 2026 16:17
@aanm aanm added the release-note/misc This PR makes changes that have no direct user impact. label Jan 6, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jan 6, 2026
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jan 6, 2026

/test

Copy link
Copy Markdown
Contributor

@Artyop Artyop left a comment

Choose a reason for hiding this comment

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

I'm not familiar with the race condition issue on multiple releases, the technical part of the retry mechanism lgtm

@aanm aanm added this pull request to the merge queue Jan 7, 2026
Merged via the queue into main with commit 183559d Jan 7, 2026
86 checks passed
@aanm aanm deleted the pr/fix-release-helm branch January 7, 2026 10:42
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/misc This PR makes changes that have no direct user impact.

Projects

No open projects
Status: Released

Development

Successfully merging this pull request may close these issues.

3 participants