On Tue, Feb 9, 2010 at 9:48 AM, Willy Tarreau <[email protected]> wrote: > On Mon, Feb 08, 2010 at 12:35:32PM -0800, Paul Hirose wrote: >> The error earlier reported during -dev7 about not compiling with >> DEBUG_FULL is still present in rc1 and ss-20100208. As that msg >> indicated, changing it f rom req to rep allowed the compile. > > grrr, it seems I lost the patch. Care to resend it please ?
It wasn't a diff-type patch so much as a sentence describing it :) In src/session.c around line 736 in function process_store_rules() you just have "req" in the DPRINTF statement, and it should be a "rep". I found mention of it at http://marc.info/?l=haproxy&m=126454210721922&w=2 739,743c739,743 < rep, < rep->rex, rep->wex, < rep->flags, < rep->l, < rep->analysers); --- > req, > req->rex, req->wex, > req->flags, > req->l, > req->analysers); > Not necessarily. DEBUG_* macros are not used to debug traffic but for > developers to more easily debug the integration of their code in haproxy. > There is no specification of what macro does what, and it can change with > time. Yah, I kinda thought it might be. After looking at it, it really wasn't understandable to me easily, so I let it go. The basic -d -V command line gave me some info, but all in all we're still tracking it down. > If you get the chance to find a user who can reliably reproduce the > issue, do not hesitate, start a tcpdump capture and you'll be fixed ! That'll be the next step. Alas, it's not reproducible on demand, but happens "reliably" after a while. We know it will happen, just not quite when or why or how :) Just depends on how big a tcpdump we want :) Thank you, PH
