Skip to content

Commit

Permalink
fix: installation on eks
Browse files Browse the repository at this point in the history
There is an issue installing Rancher Turtles on EKS using the chart as
the kubernetes version in EKS doesn't follow semver. For example, EKS
will return a version like `v1.26.12-eks-5e0fdde` and this means that we
then try and pull the `docker.io/rancher/kubectl:v1.26.12-eks-5e0fdde`
image when performing the cleanup tasks.

The function to strip the suffix from the kubernetes version has been
updated to handle EKS, AKS and GKE.

Signed-off-by: Richard Case <[email protected]>
  • Loading branch information
richardcase committed Jan 17, 2024
1 parent f5ada58 commit f9c6bf3
Showing 1 changed file with 15 additions and 2 deletions.
17 changes: 15 additions & 2 deletions charts/rancher-turtles/templates/_helpers.tpl
Original file line number Diff line number Diff line change
@@ -1,9 +1,22 @@
{{/*
This removes the part after the + in the kubernetes version string.
v1.27.4+k3s1 -> v1.27.4
v1.26.12-eks-5e0fdde -> v1.26.12
v1.26.12-gke.1 -> v1.26.12
v1.28.0 -> v1.28.0
*/}}
{{- define "strippedKubeVersion" -}}
{{- $parts := split "+" .Capabilities.KubeVersion.Version -}}
{{- print $parts._0 -}}
{{- if (.Capabilities.KubeVersion.Version | contains "-eks-") -}}
{{- $parts := split "-eks-" .Capabilities.KubeVersion.Version -}}
{{- print $parts._0 -}}
{{- else if (.Capabilities.KubeVersion.Version | contains "-gke.") -}}
{{- $parts := split "-gke." .Capabilities.KubeVersion.Version -}}
{{- print $parts._0 -}}
{{- else if (.Capabilities.KubeVersion.Version | contains "-aks") -}}
{{- $parts := split "-aks" .Capabilities.KubeVersion.Version -}}
{{- print $parts._0 -}}
{{- else -}}
{{- $parts := split "+" .Capabilities.KubeVersion.Version -}}
{{- print $parts._0 -}}
{{- end -}}
{{- end -}}

0 comments on commit f9c6bf3

Please sign in to comment.