-
Notifications
You must be signed in to change notification settings - Fork 0
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
new test: retry policy #13
Comments
Can you describe:
|
two things to check:
|
Done in 894c126 |
there's only one request there, so we're not sure it would try the backend that is down right away, and the test does not verify in sozu's logs that one of the backends was detected as down |
Not really if we use a little trick like I did: I put the down backend in top of the
Can I use this log https://github.com/sozu-proxy/sozu/blob/e0d026a4ac856057cd23e9c5c92f2b7b1f7d7d18/lib/src/network/http.rs#L506 ? |
yes |
it would probably be useful to have a test to detect the "no more available backends" message, but for this we would need to d requests in a loop until the message appears |
The problem is when we set up an application with two backends, but only one of the backend server is actually up, the log "no more available backends" is never logged but when we set up an application with only backend down (no backend up) then that work. |
to get the "no more available backends" message, we need another test with one or more backends in the configuration, but no actual backend server is running |
Okai I'll do that |
The text was updated successfully, but these errors were encountered: