Skip to content

Fix chart install failure for Kubernetes versions with build suffixes#40

Merged
gramidt merged 3 commits intomainfrom
fix/kubeversion-prerelease-compat
Mar 27, 2026
Merged

Fix chart install failure for Kubernetes versions with build suffixes#40
gramidt merged 3 commits intomainfrom
fix/kubeversion-prerelease-compat

Conversation

@gramidt
Copy link
Copy Markdown
Member

@gramidt gramidt commented Mar 27, 2026

Description
The kubeVersion constraint >=1.24.0 rejects Kubernetes versions with build suffixes like v1.33.8-eks-3a10415. Helm treats these suffixes as semver pre-release identifiers, which compare as less than the release version. Changing to >=1.24.0-0 allows these versions to pass the constraint. This is a no-op for clusters that report clean version strings.

Testing
Verified that >=1.24.0-0 accepts both plain versions (1.33.8) and suffixed versions (1.33.8-eks-3a10415), and still correctly rejects versions below 1.24.0.

Documentation
No documentation changes needed.

@gramidt gramidt requested a review from wfernandes as a code owner March 27, 2026 14:34
@gramidt gramidt changed the title Fix chart install failure on EKS, GKE, and AKS clusters Fix chart install failure for Kubernetes versions with build suffixes Mar 27, 2026
@gramidt gramidt force-pushed the fix/kubeversion-prerelease-compat branch from 9e153c9 to f910bef Compare March 27, 2026 14:35
@gramidt gramidt force-pushed the fix/kubeversion-prerelease-compat branch from e63080b to 1e1fa2d Compare March 27, 2026 14:37
@gramidt gramidt requested a review from rhysecampbell March 27, 2026 14:44
Copy link
Copy Markdown

@rhysecampbell rhysecampbell left a comment

Choose a reason for hiding this comment

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

LGTM!

@gramidt gramidt merged commit e98c6bb into main Mar 27, 2026
2 checks passed
@gramidt gramidt deleted the fix/kubeversion-prerelease-compat branch March 27, 2026 14:46
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