Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] Added keepalive_async_fails command

Maxim Dounin
April 02, 2023 09:44PM
Hello!

On Sun, Apr 02, 2023 at 06:57:03PM +0100, J Carter wrote:

> I've also attached an example nginx.conf and test script that
> simulates the asynchronous close events.
> Two different test cases can be found within that, one with path
> /1 for single peer upstream and /2 for multi-peer.
>
> You should see 2 upstream addresses repeated in a row
> per-upstream-server in the access log by default, as it fails
> through the cached connections & next performs next upstream
> tries.
>
> Any feedback would be appreciated.
>
> # HG changeset patch
> # User jordanc.carter@outlook.com
> # Date 1680457073 -3600
> # Sun Apr 02 18:37:53 2023 +0100
> # Node ID 9ec4d7a8cdf6cdab00d09dff75fa6045f6f5533f
> # Parent 5f1d05a21287ba0290dd3a17ad501595b442a194
> Added keepalive_async_fails command to keepalive load balancer module.
> This value determines the number suspected keepalive race events
> per-upstream-try that will be tolerated before a subsequent network connection
> error is considered a true failure.

It looks like you are trying to address issues with re-trying
requests to upstream servers when errors not distinguishable from
asynchronous close events happens, as outlined in ticket #2421
(https://trac.nginx.org/nginx/ticket/2421).

I don't think that introducing another directive for this is a
good idea. Rather, I would consider modifying the behaviour to
better behave in case of such errors. In particular, the
following approaches look promising:

- Allow at most one additional try.

- Allow an additional try only if there is a single peer, so
normally request is not going to be retried at all.

- Don't use cached connections if an error considered to be an
asynchronous close event happens.

Given that since nginx 1.15.3 we have keepalive_timeout in the
upstream blocks to mitigate potential asynchronous close events,
even something simple like combination of (1) and (2) might be
good enough. With (3), things are going to be as correct as it
can be.

[...]

--
Maxim Dounin
http://mdounin.ru/
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
https://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] Added keepalive_async_fails command

J Carter 395 April 02, 2023 01:58PM

Re: [PATCH] Added keepalive_async_fails command

J Carter 127 April 02, 2023 02:32PM

Re: [PATCH] Added keepalive_async_fails command

Maxim Dounin 101 April 02, 2023 09:44PM

Re: [PATCH] Added keepalive_async_fails command

J Carter 216 April 03, 2023 01:16AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 150
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready