Skip to content

Commit 233f106

Browse files
authored
Merge pull request #7305 from tstromberg/conntrack-path
none: Use LookPath to verify conntrack install
2 parents 1e70286 + 1f31d25 commit 233f106

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

cmd/minikube/cmd/start.go

+2-4
Original file line numberDiff line numberDiff line change
@@ -836,10 +836,8 @@ func validateFlags(cmd *cobra.Command, drvName string) {
836836
// conntrack is required starting with kubernetes 1.18, include the release candidates for completion
837837
version, _ := util.ParseKubernetesVersion(getKubernetesVersion(nil))
838838
if version.GTE(semver.MustParse("1.18.0-beta.1")) {
839-
err := exec.Command("conntrack").Run()
840-
if err != nil {
841-
exit.WithCodeT(exit.Config, "The none driver requires conntrack to be installed for kubernetes version {{.k8sVersion}}", out.V{"k8sVersion": version.String()})
842-
839+
if _, err := exec.LookPath("conntrack"); err != nil {
840+
exit.WithCodeT(exit.Config, "Sorry, Kubernetes v{{.k8sVersion}} requires conntrack to be installed in root's path", out.V{"k8sVersion": version.String()})
843841
}
844842
}
845843
}

0 commit comments

Comments
 (0)