Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[nginx] Ignore Ingress Annotation broken #407

Closed
InAnimaTe opened this issue Mar 8, 2017 · 2 comments
Closed

[nginx] Ignore Ingress Annotation broken #407

InAnimaTe opened this issue Mar 8, 2017 · 2 comments

Comments

@InAnimaTe
Copy link

On version 0.8.3, the Ingress annotation ignore option (added with this pr) doesn't work.

Setting kubernetes.io/ingress.class to something other than nginx for my ingress does not stop it from being accessible through the load balancer.

I0308 20:40:39.378971       1 controller.go:157] Ignoring add for ingress hastebin based on annotation kubernetes.io/ingress.class
I0308 20:40:42.377270       1 controller.go:420] updating service hastebin with new named port mappings
I0308 20:40:52.403862       1 command.go:76] change in configuration detected. Reloading...
192.168.1.149 - [192.168.1.149] - - [08/Mar/2017:20:40:59 +0000] "GET / HTTP/1.1" 200 920 "-" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36" 412 0.087 10.2.0.57:7777 920 0.087 200
192.168.1.149 - [192.168.1.149] - - [08/Mar/2017:20:40:59 +0000] "GET /solarized_dark.css HTTP/1.1" 200 532 "http://hastebin.example.io/" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36" 381 0.011 10.2.0.57:7777 532 0.011 200
192.168.1.149 - [192.168.1.149] - - [08/Mar/2017:20:40:59 +0000] "GET /application.css HTTP/1.1" 200 1016 "http://hastebin.example.io/" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36" 378 0.012 10.2.0.57:7777 1016 0.012 200
192.168.1.149 - [192.168.1.149] - - [08/Mar/2017:20:40:59 +0000] "GET /application.min.js HTTP/1.1" 

...snip...
@aledbf
Copy link
Member

aledbf commented Mar 8, 2017

@InAnimaTe please update to the latest version https://github.com/kubernetes/ingress/releases/tag/nginx-0.9.0-beta.2
This was fixed in #362

@InAnimaTe
Copy link
Author

Ahh thanks. In the process of moving. I'll close this for now and if I have any more issues in the future, open a fresh bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants