Basic Auth Annotations Changed?

[plainlystated]

I have a manifest for ingress with basic auth, which worked on an older version of the helm chart. I am trying to apply that with the latest helm stable (1.6.3), and the auth annotations have no effect. Content is served by nginx, but without requiring basic auth.

I'm sure I'm just doing something dumb... but I've been over the docs several times, and I don't see what I'm doing wrong. I also don't see an errors from the nginx pod, even when turning debugging up to 5.

  annotations:
    kubernetes.io/ingress.class: nginx
    nginx.ingress.kubernetes.io/auth-type: basic
    nginx.ingress.kubernetes.io/auth-secret: basic-auth
    nginx.ingress.kubernetes.io/auth-realm: 'Authentication Required'

I've also tried with the annotations omitting "nginx." (eg ingress.kubernetes.io/auth-type: basic). I know you can specify the annotation prefix, but I don't see any way to query what the correct prefix is from a running pod/deployment.

I have the basic-auth secret in both the nginx-ingress namespace, and the namespace of my ingress resource.

I'm on GKE, with the latest stable helm chart:

nginx@nginx-ingress-nginx-ingress-7f4fdc49d9-59pnl:/$ /nginx-ingress --version
Version=1.6.3 GitCommit=b9378d56

[aledbf]

/nginx-ingress --version
Version=1.6.3 GitCommit=b9378d56

That is for a different ingress controller https://github.com/nginxinc/kubernetes-ingress

The helm chart for ingress-nginx is stable/nginx-ingress

[plainlystated]

Thanks, I knew I was doing something dumb!

I switched to using the stable chart, and I'm good now.

Will the official chart move at some point? I thought "stable" was being deprecated this year.

[aledbf]

@plainlystated the work yo move the chart to this repository already started #5161

[plainlystated]

Thanks for the update! I'll keep an eye on that.