On Wed, 23 Sep 2009, Willy Tarreau wrote:

Hi Krzysztof,
Hi Willy!

On Wed, Sep 23, 2009 at 02:15:41AM +0200, Krzysztof Oledzki wrote:


On Thu, 17 Sep 2009, Willy Tarreau wrote:

Hi Krzysztof,

On Thu, Sep 17, 2009 at 04:21:20PM +0200, Krzysztof Oledzki wrote:
(...)
Strange, I must have overlooked something, probably because I don't use
pure tcp mode so my tests were rather limited, even too limited. :(

No problem, what matters is that it's improving :-)

Yep, silly overlook:

--- a/src/checks.c
+++ b/src/checks.c
@@ -523,6 +523,7 @@ static int event_srv_chk_w(int fd)

                        /* good TCP connection is enough */
                        s->result |= SRV_CHK_RUNNING;
+                       set_server_check_status(s, HCHK_STATUS_L4OK);
                        goto out_wakeup;
                }
        }

Good catch !

I'll do some more tests to make sure all cases are now really covered and
send the final patch, I hope. ;) Then we may talk about more improvements.
:)

That's perfect. I was about to issue -dev3 last evening but found a bug
which I have to fix before. I think I'll be able to release it this week,
so if you think you can have the patch ready to be included in the same
version, please raise your hand and I'll sync with you.

I should be able to get into this task at about midnight so if you are not releasing it today I will fit into the release window. ;)

Best regards,

                                Krzysztof Olędzki

Reply via email to