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

unnecessary backend reload #63

Closed
gianrubio opened this issue Dec 15, 2016 · 3 comments · Fixed by #64
Closed

unnecessary backend reload #63

gianrubio opened this issue Dec 15, 2016 · 3 comments · Fixed by #64
Assignees

Comments

@gianrubio
Copy link
Contributor

My backend are constantly reloading, I realised that only the ip order are changed in the templates.

Logs

I1215 12:33:49.273475       7 nginx.go:209] NGINX configuration diff
I1215 12:33:49.273665       7 nginx.go:210] --- /tmp/a580613529	2016-12-15 12:33:49.000000000 +0000
+++ /tmp/b005786148	2016-12-15 12:33:49.000000000 +0000
@@ -179,10 +179,10 @@
         ssl_certificate_key                     /ingress-controller/ssl/*.pem;
         more_set_headers                        "Strict-Transport-Security: max-age=15724800; preload";
         location / {
+           allow 213.15.3.1/32;
            allow 52.18.6.14/32;
            allow 94.28.18.253/32;
            allow 52.21.16.13/32;
-           allow 213.15.3.14/32;
            deny all;
             # enforce ssl on server side
             if ($scheme = http) {
I1215 12:33:49.295139       7 queue.go:54] queuing item &TypeMeta{Kind:,APIVersion:,}
I1215 12:33:49.295489       7 event.go:217] Event(api.ObjectReference{Kind:"Ingress", Namespace:"*", Name:"*", UID:"c69e3e6b-b888-11e6-8f0a-028561e17cf1", APIVersion:"extensions", ResourceVersion:"4223051", FieldPath:""}): type: 'Normal' reason: 'UPDATE' Ingress */*
I1215 12:33:49.304545       7 controller.go:375] ingress backend successfully reloaded...
I1215 12:33:49.664964       7 nginx.go:209] NGINX configuration diff
I1215 12:33:49.665150       7 nginx.go:210] --- /tmp/a028406237	2016-12-15 12:33:49.000000000 +0000
+++ /tmp/b445085592	2016-12-15 12:33:49.000000000 +0000
@@ -179,10 +179,10 @@
         ssl_certificate_key                     /ingress-controller/ssl/*.pem;
         more_set_headers                        "Strict-Transport-Security: max-age=15724800; preload";
         location / {
-           allow 213.15.3.1/32;
-           allow 52.18.6.14/32;
            allow 94.28.18.253/32;
            allow 52.21.16.13/32;
+           allow 213.15.3.14/32;
+           allow 52.18.6.14/32;
            deny all;
             # enforce ssl on server side
             if ($scheme = http) {
I1215 12:33:49.696796       7 controller.go:375] ingress backend successfully reloaded...
@mfojtak
Copy link

mfojtak commented Jun 20, 2017

this bug has been removed in beta.6 but appeared again in gcr.io/google_containers/nginx-ingress-controller:0.9.0-beta.6
It makes the nginx ingress controller useless

@gianrubio
Copy link
Contributor Author

@aledbf I got the same issue as @mfojtak described. Does #862 fix this issue?

@aledbf
Copy link
Member

aledbf commented Jun 20, 2017

@gianrubio yes. That is not included in beta 8. Please use quay.io/aledbf/nginx-ingress-controller:0.146

devent added a commit to devent/sscontrol-osgi that referenced this issue May 7, 2018
hnrytrn pushed a commit to hnrytrn/ingress-nginx that referenced this issue Jun 20, 2018
haoqing0110 referenced this issue in stolostron/management-ingress Mar 5, 2021
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

Successfully merging a pull request may close this issue.

3 participants