kubernetes
/
minikube
mirror of https://github.com/kubernetes/minikube.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Merge pull request from Dean-Coakley/add-ingress-noneDriver-warn 

Add warning for ingress addon enabled with driver of none
pull/8956/head
Medya Ghazizadeh 2020-08-10 13:17:54 -07:00 committed by GitHub
parent 108d2fa149 9c23485dfb
commit a13e5e5512
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 3 deletions
pkg/addons

11
pkg/addons/addons.go
View File

@ -141,15 +141,20 @@ func enableOrDisableAddon(cc *config.ClusterConfig, name string, val string) err
} }
} }
// to match both ingress and ingress-dns adons // to match both ingress and ingress-dns addons
if strings.HasPrefix(name, "ingress") && enable && driver.IsKIC(cc.Driver) && runtime.GOOS != "linux" { if strings.HasPrefix(name, "ingress") && enable {
exit.UsageT(`Due to {{.driver_name}} networking limitations on {{.os_name}}, {{.addon_name}} addon is not supported for this driver. if driver.IsKIC(cc.Driver) && runtime.GOOS != "linux" {
exit.UsageT(`Due to networking limitations of driver {{.driver_name}} on {{.os_name}}, {{.addon_name}} addon is not supported.
Alternatively to use this addon you can use a vm-based driver: Alternatively to use this addon you can use a vm-based driver:
'minikube start --vm=true' 'minikube start --vm=true'
To track the update on this work in progress feature please check: To track the update on this work in progress feature please check:
https://github.com/kubernetes/minikube/issues/7332`, out.V{"driver_name": cc.Driver, "os_name": runtime.GOOS, "addon_name": name}) https://github.com/kubernetes/minikube/issues/7332`, out.V{"driver_name": cc.Driver, "os_name": runtime.GOOS, "addon_name": name})
} else if driver.BareMetal(cc.Driver) {
exit.UsageT(`Due to networking limitations of driver {{.driver_name}}, {{.addon_name}} addon is not supported. Try using a different driver.`,
out.V{"driver_name": cc.Driver, "addon_name": name})
}
} }
if strings.HasPrefix(name, "istio") && enable { if strings.HasPrefix(name, "istio") && enable {