Skip to content

Commit

Permalink
accept kubernetes version without patch version
Browse files Browse the repository at this point in the history
  • Loading branch information
JudahNour committed May 23, 2023
1 parent 8eb0532 commit a81e4f7
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 2 deletions.
13 changes: 11 additions & 2 deletions cmd/minikube/cmd/start.go
Original file line number Diff line number Diff line change
Expand Up @@ -1757,8 +1757,17 @@ $ minikube config unset kubernetes-version`)
} else if strings.EqualFold(strings.ToLower(paramVersion), "latest") || strings.EqualFold(strings.ToLower(paramVersion), "newest") {
paramVersion = constants.NewestKubernetesVersion
}

nvs, err := semver.Make(strings.TrimPrefix(paramVersion, version.VersionPrefix))
kubernetesSemver := strings.TrimPrefix(paramVersion, version.VersionPrefix)
majorMinorOnly := regexp.MustCompile(`^(?P<major>0|[1-9]\d*)\.(?P<minor>0|[1-9]\d*)$`)
if majorMinorOnly.MatchString(kubernetesSemver) {
for _, k := range constants.ValidKubernetesVersions {
if strings.HasPrefix(k, fmt.Sprintf("v%s", kubernetesSemver)) {
kubernetesSemver = strings.TrimPrefix(k, version.VersionPrefix)
break
}
}
}
nvs, err := semver.Make(kubernetesSemver)
if err != nil {
exit.Message(reason.Usage, `Unable to parse "{{.kubernetes_version}}": {{.error}}`, out.V{"kubernetes_version": paramVersion, "error": err})
}
Expand Down
10 changes: 10 additions & 0 deletions cmd/minikube/cmd/start_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,16 @@ func TestGetKubernetesVersion(t *testing.T) {
paramVersion: "v1.16.0",
cfg: &cfg.ClusterConfig{KubernetesConfig: cfg.KubernetesConfig{KubernetesVersion: "v1.15.0"}},
},
{
description: "kubernetes-version without patch version",
expectedVersion: "v1.16.15",
paramVersion: "v1.16",
},
{
description: "kubernetes-version without patch version",
expectedVersion: "v1.16.15",
paramVersion: "1.16",
},
{
description: "kubernetes-version given as 'stable', no config",
expectedVersion: constants.DefaultKubernetesVersion,
Expand Down

0 comments on commit a81e4f7

Please sign in to comment.