Skip to content

Commit

Permalink
turn off Istio isolation for basic auth (#3675)
Browse files Browse the repository at this point in the history
* turn off Istio isolation for basic auth

* set param in config instead
  • Loading branch information
kunmingg authored and k8s-ci-robot committed Jul 17, 2019
1 parent d0c11dd commit e6944f3
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
3 changes: 3 additions & 0 deletions bootstrap/config/overlays/istio/kfctl_default.yaml
Expand Up @@ -20,6 +20,9 @@ spec:
istio-install:
- name: namespace
value: istio-system
istio:
- name: clusterRbacConfig
value: "ON"
iap-ingress:
- name: namespace
value: istio-system
Expand Down
3 changes: 3 additions & 0 deletions bootstrap/pkg/kfapp/gcp/gcp.go
Expand Up @@ -1707,6 +1707,9 @@ func (gcp *Gcp) Generate(resources kftypes.ResourceEnum) error {
if err := gcp.kfDef.SetApplicationParameter("basic-auth-ingress", "project", gcp.kfDef.Spec.Project); err != nil {
return errors.WithStack(err)
}
if err := gcp.kfDef.SetApplicationParameter("istio", "clusterRbacConfig", "OFF"); err != nil {
return errors.WithStack(err)
}
} else {
if err := gcp.kfDef.SetApplicationParameter("iap-ingress", "ipName", gcp.kfDef.Spec.IpName); err != nil {
return errors.WithStack(err)
Expand Down

0 comments on commit e6944f3

Please sign in to comment.