Skip to content

Commit f17b608

Browse files
authored
Merge pull request kubernetes#81622 from neolit123/kubeadm-remove-blang
kubeadm: do not use github.com/blang/semver in app/preflight
2 parents 5c68a23 + 0a0f6f8 commit f17b608

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

cmd/kubeadm/app/preflight/BUILD

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@ go_library(
3131
"//staging/src/k8s.io/apimachinery/pkg/util/sets:go_default_library",
3232
"//staging/src/k8s.io/apimachinery/pkg/util/version:go_default_library",
3333
"//vendor/github.com/PuerkitoBio/purell:go_default_library",
34-
"//vendor/github.com/blang/semver:go_default_library",
3534
"//vendor/github.com/pkg/errors:go_default_library",
3635
"//vendor/k8s.io/klog:go_default_library",
3736
"//vendor/k8s.io/utils/exec:go_default_library",

cmd/kubeadm/app/preflight/checks.go

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ import (
3535
"time"
3636

3737
"github.com/PuerkitoBio/purell"
38-
"github.com/blang/semver"
3938
"github.com/pkg/errors"
4039
netutil "k8s.io/apimachinery/pkg/util/net"
4140
"k8s.io/apimachinery/pkg/util/sets"
@@ -65,7 +64,7 @@ const (
6564
)
6665

6766
var (
68-
minExternalEtcdVersion = semver.MustParse(kubeadmconstants.MinExternalEtcdVersion)
67+
minExternalEtcdVersion = versionutil.MustParseSemantic(kubeadmconstants.MinExternalEtcdVersion)
6968
)
7069

7170
// Error defines struct for communicating error messages generated by preflight checks
@@ -725,12 +724,12 @@ func (evc ExternalEtcdVersionCheck) Check() (warnings, errorList []error) {
725724
continue
726725
}
727726

728-
etcdVersion, err := semver.Parse(resp.Etcdserver)
727+
etcdVersion, err := versionutil.ParseSemantic(resp.Etcdserver)
729728
if err != nil {
730729
errorList = append(errorList, errors.Wrapf(err, "couldn't parse external etcd version %q", resp.Etcdserver))
731730
continue
732731
}
733-
if etcdVersion.LT(minExternalEtcdVersion) {
732+
if etcdVersion.LessThan(minExternalEtcdVersion) {
734733
errorList = append(errorList, errors.Errorf("this version of kubeadm only supports external etcd version >= %s. Current version: %s", kubeadmconstants.MinExternalEtcdVersion, resp.Etcdserver))
735734
continue
736735
}

0 commit comments

Comments
 (0)