Re: [PATCH] implement "track backend/server"

On Sun, 10 Feb 2008, Willy Tarreau wrote: <CUT>
> Just a few comments :
> - when you parse the "track" keyword, I think that the name of the proxy
> and server should not be checked immediately, but deffered. This would
> allow to track independently of the declaration order. That's what is
> done for use_backend for instance. The change is not necessarily hard,
> it basically consists in moving the assignment code further (around
> line 3000 in cfg_parse.c), and adding a check to ensure that
> srv->tracked->tracked == NULL to ensure there's no loop and that we're
> not chaining tracking.

Done.

> - there's one extra comma in the chunk below :
Fixed.

Just sent a new version. Sorry for the wrong subject - my regex didn't handle it properly - need to fix it. :|

Best regards,

                                 Krzysztof Olêdzki Received on 2008/02/18 01:37