Re: SSL handshake failure

2013-02-09 Thread Willy Tarreau
On Sat, Feb 09, 2013 at 08:45:50PM +0400, Samat Galimov wrote: > Can I help in any way? Not yes I think, I'll check with Emeric if we can find other locations where errors might remain uncaught. Thanks! Willy

Re: SSL handshake failure

2013-02-09 Thread Samat Galimov
Can I help in any way? On Fri, Feb 8, 2013 at 11:29 PM, Willy Tarreau wrote: > On Fri, Feb 08, 2013 at 11:27:31PM +0400, Samat Galimov wrote: > > You are right. > > After I start haproxy with patch it establishes first connection > normally, > > as it should. > > All following connections are r

Re: [PATCH 1/3] MEDIUM: New cli option -Ds for systemd compatibility

2013-02-09 Thread Willy Tarreau
On Sat, Feb 09, 2013 at 11:45:57AM +0100, Marc-Antoine Perennou wrote: > Maybe this can help you understand why it works: > > Before the download: > haproxy 6090 root4u IPv4 53819 0t0 TCP > *:http (LISTEN) > haproxy 6092 root4u IPv4 53819

Re: [PATCH 1/3] MEDIUM: New cli option -Ds for systemd compatibility

2013-02-09 Thread Marc-Antoine Perennou
On 9 February 2013 11:06, Willy Tarreau wrote: > Hi, > > On Sat, Feb 09, 2013 at 10:44:04AM +0100, Marc-Antoine Perennou wrote: > > I just made a simple test, running a webserver serving a big file > locally, > > using haproxy, > > my wrapper and systemd service. I started a download and during t

Re: [PATCH 2/3] MEDIUM: add haproxy-systemd-wrapper

2013-02-09 Thread Willy Tarreau
On Sat, Feb 09, 2013 at 10:47:23AM +0100, Marc-Antoine Perennou wrote: > I'll do even simpler, when submitting V2 of this patchset with the EINTR > issue, > I'll basically forard every arg given to the wrapper to all the children > themselves. > The tool is not really meant to be used by the user h

Re: [PATCH 1/3] MEDIUM: New cli option -Ds for systemd compatibility

2013-02-09 Thread Willy Tarreau
Hi, On Sat, Feb 09, 2013 at 10:44:04AM +0100, Marc-Antoine Perennou wrote: > I just made a simple test, running a webserver serving a big file locally, > using haproxy, > my wrapper and systemd service. I started a download and during this > download, > reloaded haproxy. I using nbproc = 2. > What

Re: [PATCH 2/3] MEDIUM: add haproxy-systemd-wrapper

2013-02-09 Thread Marc-Antoine Perennou
I'll do even simpler, when submitting V2 of this patchset with the EINTR issue, I'll basically forard every arg given to the wrapper to all the children themselves. The tool is not really meant to be used by the user himself though, but rather to be launched by the systemd service. Is SIGUSR2 ok he

Re: [PATCH 1/3] MEDIUM: New cli option -Ds for systemd compatibility

2013-02-09 Thread Marc-Antoine Perennou
On 9 February 2013 09:45, Willy Tarreau wrote: > On Fri, Feb 08, 2013 at 03:58:46PM +0100, Marc-Antoine Perennou wrote: > > @@ -1493,8 +1499,13 @@ int main(int argc, char **argv) > > px = px->next; > > } > > > > - if (proc == global.nbproc) > > +

Re: [PATCH 2/3] MEDIUM: add haproxy-systemd-wrapper

2013-02-09 Thread Willy Tarreau
On Fri, Feb 08, 2013 at 03:58:47PM +0100, Marc-Antoine Perennou wrote: > +static void usage(const char *progname) > +{ > + fprintf(stderr, "Usage: %s [-f cfgfile] [-p pidfile]\n", progname); Here I think it would be worth supporting most of the other command line arguments. For instance, -L is

Re: [PATCH 1/3] MEDIUM: New cli option -Ds for systemd compatibility

2013-02-09 Thread Willy Tarreau
On Fri, Feb 08, 2013 at 03:58:46PM +0100, Marc-Antoine Perennou wrote: > @@ -1493,8 +1499,13 @@ int main(int argc, char **argv) > px = px->next; > } > > - if (proc == global.nbproc) > + if (proc == global.nbproc) { > +

Re: HAProxy and systemd compatibility

2013-02-09 Thread Willy Tarreau
Hi Marc-Antoine, On Fri, Feb 08, 2013 at 03:58:45PM +0100, Marc-Antoine Perennou wrote: > Hi, > > Currently, to reload haproxy configuration, you have to use "-sf". > > Systemd philosophy is for the daemon not to fork by themselves, but rather let > the init process do it for them. > > My first