New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[eks] [request]: Use correct SemVer for Kubernetes version #1404
Comments
With the EKS EKS v1.21
EKS v1.20
|
The version notation change introduced in the release of EKS v1.21 (removal of "eks") has since been addressed. The 1.21 version now looks similar to how it was with shown for previous EKS versions. However the original issue raised here (usage of pre-release tag) has not been addressed yet but we are closely monitoring the 馃憤 reaction to the issue here to help with our prioritization decision. |
@georgejohnis with EKS 1.22 issue returned.
|
@z0rc I think that's the "supported" behaviour and they're looking for enough community pressure to prioritise alignment to standard semver formatting. From this Helm issue it sounds like there's a non-trivial amount of effort to change it. Should make the Kubernetes cluster conformance suites fail on invalid semver as it really becomes a burden of Helm chart authors who are probably too busy shipping features to search for this issue. |
I'm not sure why the build metadata part of the SemVer spec isn't being utilised here? E.g |
@stevehipwell while unfortunate, I don't think it matters at this point for 2 reasons:
|
@nfisher I'd strongly disagree and say that it still matters, but I'm not expecting a change any time soon. This "decision" to not follow SemVer spec literally makes no sense given the high level Kubernetes version based architecture and the fact that there is a documented pattern for carrying extra version metadata. |
Community Note
Tell us about your request
I'd like the EKS Kubernetes version to be a valid Semver 2 version, meaning no pre-release segment if it's not a pre-release.
Which service(s) is this request for?
EKS
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
The current Kubernetes version for EKS is an incorrect pre-release SemVer version (e.g.
v1.18.9-eks-d1db3c
) and causes any automation expecting a valid SemVer to fail. It looks like this is because you're trying to store two pieces of data in a variable only capable of storing one.This means that Helm charts which use
.Capabilities.KubeVersion.Version
for conditional logic either need to provide a static override or manually sanitise the version to remove the pre-release segment.Releated issue prometheus-community/helm-charts#1071.
Are you currently working around this issue?
For Helm this is only possible in some scenarios when the author of the chart has already come across this issue and has enabled the version to be set manually or sanitises the version to ignore pre-release segments.
Additional context
n/a
Attachments
If you think you might have additional information that you'd like to include via an attachment, please do - we'll take a look. (Remember to remove any personally-identifiable information.)
The text was updated successfully, but these errors were encountered: