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

Backoff not reset on new ingress resources #599

Closed
bdirito opened this issue Sep 10, 2018 · 3 comments
Closed

Backoff not reset on new ingress resources #599

bdirito opened this issue Sep 10, 2018 · 3 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@bdirito
Copy link

bdirito commented Sep 10, 2018

When trying something new that doesnt work (in my case a redirect config that is creating a loop). The alb ingress controller fails and goes into a backoff. This is good. However when you then edit the ingress resource to try something new you are still in that same backoff loop. You have to wait for the timeout of the previous backoff to finish before you can try your changes. If those then fail you increment again and will now have to wait for backoff again before checking anything again.

Current best workarround is to delete the alb ingress controller pod

This is annoying for development as it slows down the pickup of the new rules.

@bigkraig bigkraig added the kind/feature Categorizes issue or PR as related to a new feature. label Sep 10, 2018
@NickLarsenNZ
Copy link

NickLarsenNZ commented Sep 19, 2018

It seems like the Backoff times are hardcoded in Kubernetes as described here: kubernetes/kubernetes#57291

I'm not sure there's anything else you can do other than restarting the pod (which doesn't impact traffic except for the broken Ingress.

I think the real solution is to probably have better validation of the Ingress config to avoid panics.

@M00nF1sh
Copy link
Collaborator

This is already resolved by #674
Will close this after I do full integration tests for all use cases

@M00nF1sh
Copy link
Collaborator

this one was fixed in master branch(we'll release new version soon)
Closing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

4 participants