Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 1 | ---------------------- |
| 2 | HAProxy how-to |
| 3 | ---------------------- |
Willy Tarreau | 15480d7 | 2014-06-19 21:10:58 +0200 | [diff] [blame] | 4 | version 1.6-dev |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 5 | willy tarreau |
Willy Tarreau | 8747b6d | 2015-03-11 23:57:23 +0100 | [diff] [blame] | 6 | 2015/03/11 |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 7 | |
| 8 | |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 9 | 1) How to build it |
| 10 | ------------------ |
| 11 | |
Willy Tarreau | 15480d7 | 2014-06-19 21:10:58 +0200 | [diff] [blame] | 12 | First, please note that this version is a development version, so in general if |
| 13 | you are not used to build from sources or if you don't have the time to track |
| 14 | very frequent updates, it is recommended that instead you switch to the stable |
| 15 | version (1.5) or follow the packaged updates provided by your software vendor |
| 16 | or Linux distribution. Most of them are taking this task seriously and are |
| 17 | doing a good job. If for any reason you'd prefer a different version than the |
| 18 | one packaged for your system, or to get some commercial support, other choices |
| 19 | are available at : |
Willy Tarreau | 869f351 | 2014-06-19 15:26:32 +0200 | [diff] [blame] | 20 | |
| 21 | http://www.haproxy.com/ |
| 22 | |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 23 | To build haproxy, you will need : |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 24 | - GNU make. Neither Solaris nor OpenBSD's make work with the GNU Makefile. |
Willy Tarreau | 3543cdb | 2014-05-10 09:12:46 +0200 | [diff] [blame] | 25 | If you get many syntax errors when running "make", you may want to retry |
| 26 | with "gmake" which is the name commonly used for GNU make on BSD systems. |
Willy Tarreau | 869f351 | 2014-06-19 15:26:32 +0200 | [diff] [blame] | 27 | - GCC between 2.95 and 4.8. Others may work, but not tested. |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 28 | - GNU ld |
| 29 | |
| 30 | Also, you might want to build with libpcre support, which will provide a very |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 31 | efficient regex implementation and will also fix some badness on Solaris' one. |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 32 | |
| 33 | To build haproxy, you have to choose your target OS amongst the following ones |
| 34 | and assign it to the TARGET variable : |
| 35 | |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 36 | - linux22 for Linux 2.2 |
| 37 | - linux24 for Linux 2.4 and above (default) |
| 38 | - linux24e for Linux 2.4 with support for a working epoll (> 0.21) |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 39 | - linux26 for Linux 2.6 and above |
Willy Tarreau | 869f351 | 2014-06-19 15:26:32 +0200 | [diff] [blame] | 40 | - linux2628 for Linux 2.6.28, 3.x, and above (enables splice and tproxy) |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 41 | - solaris for Solaris 8 or 10 (others untested) |
Willy Tarreau | 869f351 | 2014-06-19 15:26:32 +0200 | [diff] [blame] | 42 | - freebsd for FreeBSD 5 to 10 (others untested) |
Willy Tarreau | 8624cab | 2013-04-02 08:17:43 +0200 | [diff] [blame] | 43 | - osx for Mac OS/X |
Willy Tarreau | 3b8e979 | 2012-11-22 00:43:09 +0100 | [diff] [blame] | 44 | - openbsd for OpenBSD 3.1 to 5.2 (others untested) |
Willy Tarreau | 50abe30 | 2014-04-02 20:44:43 +0200 | [diff] [blame] | 45 | - aix51 for AIX 5.1 |
Willy Tarreau | 7dec965 | 2012-06-06 16:15:03 +0200 | [diff] [blame] | 46 | - aix52 for AIX 5.2 |
Yitzhak Sapir | 3208731 | 2009-06-14 18:27:54 +0200 | [diff] [blame] | 47 | - cygwin for Cygwin |
Willy Tarreau | 869f351 | 2014-06-19 15:26:32 +0200 | [diff] [blame] | 48 | - generic for any other OS or version. |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 49 | - custom to manually adjust every setting |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 50 | |
| 51 | You may also choose your CPU to benefit from some optimizations. This is |
| 52 | particularly important on UltraSparc machines. For this, you can assign |
| 53 | one of the following choices to the CPU variable : |
| 54 | |
| 55 | - i686 for intel PentiumPro, Pentium 2 and above, AMD Athlon |
| 56 | - i586 for intel Pentium, AMD K6, VIA C3. |
| 57 | - ultrasparc : Sun UltraSparc I/II/III/IV processor |
Willy Tarreau | 817dad5 | 2014-07-10 20:24:25 +0200 | [diff] [blame] | 58 | - native : use the build machine's specific processor optimizations. Use with |
| 59 | extreme care, and never in virtualized environments (known to break). |
| 60 | - generic : any other processor or no CPU-specific optimization. (default) |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 61 | |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 62 | Alternatively, you may just set the CPU_CFLAGS value to the optimal GCC options |
| 63 | for your platform. |
| 64 | |
Willy Tarreau | ef7341d | 2009-04-11 19:45:50 +0200 | [diff] [blame] | 65 | You may want to build specific target binaries which do not match your native |
| 66 | compiler's target. This is particularly true on 64-bit systems when you want |
| 67 | to build a 32-bit binary. Use the ARCH variable for this purpose. Right now |
Willy Tarreau | a5899aa | 2010-11-28 07:41:00 +0100 | [diff] [blame] | 68 | it only knows about a few x86 variants (i386,i486,i586,i686,x86_64), two |
| 69 | generic ones (32,64) and sets -m32/-m64 as well as -march=<arch> accordingly. |
Willy Tarreau | ef7341d | 2009-04-11 19:45:50 +0200 | [diff] [blame] | 70 | |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 71 | If your system supports PCRE (Perl Compatible Regular Expressions), then you |
| 72 | really should build with libpcre which is between 2 and 10 times faster than |
| 73 | other libc implementations. Regex are used for header processing (deletion, |
| 74 | rewriting, allow, deny). The only inconvenient of libpcre is that it is not |
| 75 | yet widely spread, so if you build for other systems, you might get into |
| 76 | trouble if they don't have the dynamic library. In this situation, you should |
| 77 | statically link libpcre into haproxy so that it will not be necessary to |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 78 | install it on target systems. Available build options for PCRE are : |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 79 | |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 80 | - USE_PCRE=1 to use libpcre, in whatever form is available on your system |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 81 | (shared or static) |
| 82 | |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 83 | - USE_STATIC_PCRE=1 to use a static version of libpcre even if the dynamic |
| 84 | one is available. This will enhance portability. |
| 85 | |
Willy Tarreau | 663148c | 2012-12-12 00:38:22 +0100 | [diff] [blame] | 86 | - with no option, use your OS libc's standard regex implementation (default). |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 87 | Warning! group references on Solaris seem broken. Use static-pcre whenever |
| 88 | possible. |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 89 | |
Willy Tarreau | 64bc40b | 2011-03-23 20:00:53 +0100 | [diff] [blame] | 90 | Recent systems can resolve IPv6 host names using getaddrinfo(). This primitive |
| 91 | is not present in all libcs and does not work in all of them either. Support in |
| 92 | glibc was broken before 2.3. Some embedded libs may not properly work either, |
| 93 | thus, support is disabled by default, meaning that some host names which only |
| 94 | resolve as IPv6 addresses will not resolve and configs might emit an error |
| 95 | during parsing. If you know that your OS libc has reliable support for |
| 96 | getaddrinfo(), you can add USE_GETADDRINFO=1 on the make command line to enable |
| 97 | it. This is the recommended option for most Linux distro packagers since it's |
| 98 | working fine on all recent mainstream distros. It is automatically enabled on |
| 99 | Solaris 8 and above, as it's known to work. |
| 100 | |
Willy Tarreau | 3543cdb | 2014-05-10 09:12:46 +0200 | [diff] [blame] | 101 | It is possible to add native support for SSL using the GNU makefile, by passing |
| 102 | "USE_OPENSSL=1" on the make command line. The libssl and libcrypto will |
| 103 | automatically be linked with haproxy. Some systems also require libz, so if the |
| 104 | build fails due to missing symbols such as deflateInit(), then try again with |
| 105 | "ADDLIB=-lz". |
Willy Tarreau | d450881 | 2012-09-10 09:07:41 +0200 | [diff] [blame] | 106 | |
Lukas Tribus | 3fe9f1e | 2013-05-19 16:28:17 +0200 | [diff] [blame] | 107 | To link OpenSSL statically against haproxy, build OpenSSL with the no-shared |
| 108 | keyword and install it to a local directory, so your system is not affected : |
| 109 | |
| 110 | $ export STATICLIBSSL=/tmp/staticlibssl |
| 111 | $ ./config --prefix=$STATICLIBSSL no-shared |
| 112 | $ make && make install_sw |
| 113 | |
Lukas Tribus | 130ddf7 | 2013-10-01 00:28:03 +0200 | [diff] [blame] | 114 | When building haproxy, pass that path via SSL_INC and SSL_LIB to make and |
| 115 | include additional libs with ADDLIB if needed (in this case for example libdl): |
Willy Tarreau | 3543cdb | 2014-05-10 09:12:46 +0200 | [diff] [blame] | 116 | |
Lukas Tribus | 130ddf7 | 2013-10-01 00:28:03 +0200 | [diff] [blame] | 117 | $ make TARGET=linux26 USE_OPENSSL=1 SSL_INC=$STATICLIBSSL/include SSL_LIB=$STATICLIBSSL/lib ADDLIB=-ldl |
Lukas Tribus | 3fe9f1e | 2013-05-19 16:28:17 +0200 | [diff] [blame] | 118 | |
William Lallemand | 82fe75c | 2012-10-23 10:25:10 +0200 | [diff] [blame] | 119 | It is also possible to include native support for ZLIB to benefit from HTTP |
| 120 | compression. For this, pass "USE_ZLIB=1" on the "make" command line and ensure |
Willy Tarreau | 418b8c0 | 2015-03-29 03:32:06 +0200 | [diff] [blame] | 121 | that zlib is present on the system. Alternatively it is possible to use libslz |
| 122 | for a faster, memory less, but slightly less efficient compression, by passing |
| 123 | "USE_SLZ=1". |
William Lallemand | 82fe75c | 2012-10-23 10:25:10 +0200 | [diff] [blame] | 124 | |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 125 | By default, the DEBUG variable is set to '-g' to enable debug symbols. It is |
| 126 | not wise to disable it on uncommon systems, because it's often the only way to |
| 127 | get a complete core when you need one. Otherwise, you can set DEBUG to '-s' to |
| 128 | strip the binary. |
| 129 | |
| 130 | For example, I use this to build for Solaris 8 : |
| 131 | |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 132 | $ make TARGET=solaris CPU=ultrasparc USE_STATIC_PCRE=1 |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 133 | |
Willy Tarreau | 83b30c1 | 2008-05-25 10:32:50 +0200 | [diff] [blame] | 134 | And I build it this way on OpenBSD or FreeBSD : |
willy tarreau | d38e72d | 2006-03-19 20:56:52 +0100 | [diff] [blame] | 135 | |
Willy Tarreau | 3543cdb | 2014-05-10 09:12:46 +0200 | [diff] [blame] | 136 | $ gmake TARGET=freebsd USE_PCRE=1 USE_OPENSSL=1 USE_ZLIB=1 |
willy tarreau | d38e72d | 2006-03-19 20:56:52 +0100 | [diff] [blame] | 137 | |
Willy Tarreau | 663148c | 2012-12-12 00:38:22 +0100 | [diff] [blame] | 138 | And on a classic Linux with SSL and ZLIB support (eg: Red Hat 5.x) : |
| 139 | |
Willy Tarreau | 817dad5 | 2014-07-10 20:24:25 +0200 | [diff] [blame] | 140 | $ make TARGET=linux26 USE_PCRE=1 USE_OPENSSL=1 USE_ZLIB=1 |
Willy Tarreau | 663148c | 2012-12-12 00:38:22 +0100 | [diff] [blame] | 141 | |
| 142 | And on a recent Linux >= 2.6.28 with SSL and ZLIB support : |
Willy Tarreau | d450881 | 2012-09-10 09:07:41 +0200 | [diff] [blame] | 143 | |
Willy Tarreau | 817dad5 | 2014-07-10 20:24:25 +0200 | [diff] [blame] | 144 | $ make TARGET=linux2628 USE_PCRE=1 USE_OPENSSL=1 USE_ZLIB=1 |
Willy Tarreau | d450881 | 2012-09-10 09:07:41 +0200 | [diff] [blame] | 145 | |
William Lallemand | 82fe75c | 2012-10-23 10:25:10 +0200 | [diff] [blame] | 146 | In order to build a 32-bit binary on an x86_64 Linux system with SSL support |
| 147 | without support for compression but when OpenSSL requires ZLIB anyway : |
Willy Tarreau | ef7341d | 2009-04-11 19:45:50 +0200 | [diff] [blame] | 148 | |
Willy Tarreau | d450881 | 2012-09-10 09:07:41 +0200 | [diff] [blame] | 149 | $ make TARGET=linux26 ARCH=i386 USE_OPENSSL=1 ADDLIB=-lz |
Willy Tarreau | ef7341d | 2009-04-11 19:45:50 +0200 | [diff] [blame] | 150 | |
Willy Tarreau | b1efede | 2014-05-09 00:44:48 +0200 | [diff] [blame] | 151 | The SSL stack supports session cache synchronization between all running |
| 152 | processes. This involves some atomic operations and synchronization operations |
| 153 | which come in multiple flavors depending on the system and architecture : |
| 154 | |
| 155 | Atomic operations : |
| 156 | - internal assembler versions for x86/x86_64 architectures |
| 157 | |
| 158 | - gcc builtins for other architectures. Some architectures might not |
| 159 | be fully supported or might require a more recent version of gcc. |
| 160 | If your architecture is not supported, you willy have to either use |
| 161 | pthread if supported, or to disable the shared cache. |
| 162 | |
| 163 | - pthread (posix threads). Pthreads are very common but inter-process |
| 164 | support is not that common, and some older operating systems did not |
| 165 | report an error when enabling multi-process mode, so they used to |
| 166 | silently fail, possibly causing crashes. Linux's implementation is |
| 167 | fine. OpenBSD doesn't support them and doesn't build. FreeBSD 9 builds |
| 168 | and reports an error at runtime, while certain older versions might |
| 169 | silently fail. Pthreads are enabled using USE_PTHREAD_PSHARED=1. |
| 170 | |
| 171 | Synchronization operations : |
| 172 | - internal spinlock : this mode is OS-independant, light but will not |
| 173 | scale well to many processes. However, accesses to the session cache |
| 174 | are rare enough that this mode could certainly always be used. This |
| 175 | is the default mode. |
| 176 | |
| 177 | - Futexes, which are Linux-specific highly scalable light weight mutexes |
| 178 | implemented in user-space with some limited assistance from the kernel. |
| 179 | This is the default on Linux 2.6 and above and is enabled by passing |
| 180 | USE_FUTEX=1 |
| 181 | |
| 182 | - pthread (posix threads). See above. |
| 183 | |
| 184 | If none of these mechanisms is supported by your platform, you may need to |
| 185 | build with USE_PRIVATE_CACHE=1 to totally disable SSL cache sharing. Then |
| 186 | it is better not to run SSL on multiple processes. |
| 187 | |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 188 | If you need to pass other defines, includes, libraries, etc... then please |
| 189 | check the Makefile to see which ones will be available in your case, and |
Willy Tarreau | 3543cdb | 2014-05-10 09:12:46 +0200 | [diff] [blame] | 190 | use the USE_* variables in the Makefile. |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 191 | |
Willy Tarreau | 97ec969 | 2010-01-28 20:52:05 +0100 | [diff] [blame] | 192 | AIX 5.3 is known to work with the generic target. However, for the binary to |
| 193 | also run on 5.2 or earlier, you need to build with DEFINE="-D_MSGQSUPPORT", |
Willy Tarreau | 869f351 | 2014-06-19 15:26:32 +0200 | [diff] [blame] | 194 | otherwise __fd_select() will be used while not being present in the libc, but |
| 195 | this is easily addressed using the "aix52" target. If you get build errors |
| 196 | because of strange symbols or section mismatches, simply remove -g from |
| 197 | DEBUG_CFLAGS. |
Willy Tarreau | 97ec969 | 2010-01-28 20:52:05 +0100 | [diff] [blame] | 198 | |
Willy Tarreau | 32e65ef | 2013-04-02 08:14:29 +0200 | [diff] [blame] | 199 | You can easily define your own target with the GNU Makefile. Unknown targets |
| 200 | are processed with no default option except USE_POLL=default. So you can very |
| 201 | well use that property to define your own set of options. USE_POLL can even be |
| 202 | disabled by setting USE_POLL="". For example : |
| 203 | |
| 204 | $ gmake TARGET=tiny USE_POLL="" TARGET_CFLAGS=-fomit-frame-pointer |
| 205 | |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 206 | |
David Carlier | b5efa01 | 2015-06-01 14:21:47 +0200 | [diff] [blame] | 207 | 1.1) DeviceAtlas Device Detection |
| 208 | --------------------------------- |
| 209 | |
| 210 | In order to add DeviceAtlas Device Detection support, you would need to download |
| 211 | the API source code from https://deviceatlas.com/deviceatlas-haproxy-module and |
| 212 | once extracted : |
| 213 | |
| 214 | $ make TARGET=<target> USE_PCRE=1 USE_DEVICEATLAS=1 DEVICEATLAS_INC=<path to the API root folder> DEVICEATLAS_LIB=<path to the API root folder> |
| 215 | |
| 216 | These are supported DeviceAtlas directives (see doc/configuration.txt) : |
| 217 | - deviceatlas-json-file <path to the DeviceAtlas JSON data file>. |
| 218 | - deviceatlas-log-level <number> (0 to 3, level of information returned by |
| 219 | the API, 0 by default). |
| 220 | - deviceatlas-property-separator <character> (character used to separate the |
| 221 | properties produced by the API, | by default). |
| 222 | |
| 223 | Sample configuration : |
| 224 | |
| 225 | global |
| 226 | deviceatlas-json-file <path to json file> |
| 227 | |
| 228 | ... |
| 229 | frontend |
| 230 | bind *:8881 |
| 231 | default_backend servers |
| 232 | http-request set-header X-DeviceAtlas-Data %[req.fhdr(User-Agent),da-csv(primaryHardwareType,osName,osVersion,browserName,browserVersion)] |
| 233 | |
| 234 | |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 235 | 2) How to install it |
| 236 | -------------------- |
| 237 | |
| 238 | To install haproxy, you can either copy the single resulting binary to the |
| 239 | place you want, or run : |
| 240 | |
| 241 | $ sudo make install |
| 242 | |
| 243 | If you're packaging it for another system, you can specify its root directory |
| 244 | in the usual DESTDIR variable. |
| 245 | |
| 246 | |
| 247 | 3) How to set it up |
| 248 | ------------------- |
| 249 | |
| 250 | There is some documentation in the doc/ directory : |
| 251 | |
| 252 | - architecture.txt : this is the architecture manual. It is quite old and |
| 253 | does not tell about the nice new features, but it's still a good starting |
| 254 | point when you know what you want but don't know how to do it. |
| 255 | |
| 256 | - configuration.txt : this is the configuration manual. It recalls a few |
| 257 | essential HTTP basic concepts, and details all the configuration file |
| 258 | syntax (keywords, units). It also describes the log and stats format. It |
| 259 | is normally always up to date. If you see that something is missing from |
Willy Tarreau | 74774c0 | 2014-04-23 00:57:08 +0200 | [diff] [blame] | 260 | it, please report it as this is a bug. Please note that this file is |
| 261 | huge and that it's generally more convenient to review Cyril Bonté's |
| 262 | HTML translation online here : |
| 263 | |
| 264 | http://cbonte.github.io/haproxy-dconv/configuration-1.5.html |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 265 | |
| 266 | - haproxy-en.txt / haproxy-fr.txt : these are the old outdated docs. You |
| 267 | should never need them. If you do, then please report what you didn't |
| 268 | find in the other ones. |
| 269 | |
| 270 | - gpl.txt / lgpl.txt : the copy of the licenses covering the software. See |
| 271 | the 'LICENSE' file at the top for more information. |
| 272 | |
| 273 | - the rest is mainly for developers. |
| 274 | |
| 275 | There are also a number of nice configuration examples in the "examples" |
| 276 | directory as well as on several sites and articles on the net which are linked |
| 277 | to from the haproxy web site. |
| 278 | |
| 279 | |
| 280 | 4) How to report a bug |
| 281 | ---------------------- |
| 282 | |
| 283 | It is possible that from time to time you'll find a bug. A bug is a case where |
| 284 | what you see is not what is documented. Otherwise it can be a misdesign. If you |
| 285 | find that something is stupidly design, please discuss it on the list (see the |
| 286 | "how to contribute" section below). If you feel like you're proceeding right |
| 287 | and haproxy doesn't obey, then first ask yourself if it is possible that nobody |
| 288 | before you has even encountered this issue. If it's unlikely, the you probably |
| 289 | have an issue in your setup. Just in case of doubt, please consult the mailing |
| 290 | list archives : |
| 291 | |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 292 | http://marc.info/?l=haproxy |
| 293 | |
| 294 | Otherwise, please try to gather the maximum amount of information to help |
| 295 | reproduce the issue and send that to the mailing list : |
| 296 | |
| 297 | haproxy@formilux.org |
| 298 | |
| 299 | Please include your configuration and logs. You can mask your IP addresses and |
| 300 | passwords, we don't need them. But it's essential that you post your config if |
| 301 | you want people to guess what is happening. |
| 302 | |
| 303 | Also, keep in mind that haproxy is designed to NEVER CRASH. If you see it die |
| 304 | without any reason, then it definitely is a critical bug that must be reported |
| 305 | and urgently fixed. It has happened a couple of times in the past, essentially |
| 306 | on development versions running on new architectures. If you think your setup |
| 307 | is fairly common, then it is possible that the issue is totally unrelated. |
| 308 | Anyway, if that happens, feel free to contact me directly, as I will give you |
| 309 | instructions on how to collect a usable core file, and will probably ask for |
| 310 | other captures that you'll not want to share with the list. |
| 311 | |
| 312 | |
| 313 | 5) How to contribute |
| 314 | -------------------- |
| 315 | |
| 316 | It is possible that you'll want to add a specific feature to satisfy your needs |
| 317 | or one of your customers'. Contributions are welcome, however I'm often very |
| 318 | picky about changes. I will generally reject patches that change massive parts |
| 319 | of the code, or that touch the core parts without any good reason if those |
| 320 | changes have not been discussed first. |
| 321 | |
| 322 | The proper place to discuss your changes is the HAProxy Mailing List. There are |
| 323 | enough skilled readers to catch hazardous mistakes and to suggest improvements. |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 324 | I trust a number of them enough to merge a patch if they say it's OK, so using |
| 325 | the list is the fastest way to get your code reviewed and merged. You can |
| 326 | subscribe to it by sending an empty e-mail at the following address : |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 327 | |
| 328 | haproxy+subscribe@formilux.org |
| 329 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 330 | If you have an idea about something to implement, *please* discuss it on the |
| 331 | list first. It has already happened several times that two persons did the same |
| 332 | thing simultaneously. This is a waste of time for both of them. It's also very |
| 333 | common to see some changes rejected because they're done in a way that will |
| 334 | conflict with future evolutions, or that does not leave a good feeling. It's |
| 335 | always unpleasant for the person who did the work, and it is unpleasant for me |
| 336 | too because I value people's time and efforts. That would not happen if these |
| 337 | were discussed first. There is no problem posting work in progress to the list, |
| 338 | it happens quite often in fact. Also, don't waste your time with the doc when |
| 339 | submitting patches for review, only add the doc with the patch you consider |
| 340 | ready to merge. |
| 341 | |
Willy Tarreau | 2ddccb7 | 2013-05-01 10:07:21 +0200 | [diff] [blame] | 342 | Another important point concerns code portability. Haproxy requires gcc as the |
| 343 | C compiler, and may or may not work with other compilers. However it's known |
| 344 | to build using gcc 2.95 or any later version. As such, it is important to keep |
| 345 | in mind that certain facilities offered by recent versions must not be used in |
| 346 | the code : |
| 347 | |
| 348 | - declarations mixed in the code (requires gcc >= 3.x) |
| 349 | - GCC builtins without checking for their availability based on version and |
| 350 | architecture ; |
| 351 | - assembly code without any alternate portable form for other platforms |
| 352 | - use of stdbool.h, "bool", "false", "true" : simply use "int", "0", "1" |
| 353 | - in general, anything which requires C99 (such as declaring variables in |
| 354 | "for" statements) |
| 355 | |
| 356 | Since most of these restrictions are just a matter of coding style, it is |
| 357 | normally not a problem to comply. |
| 358 | |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 359 | If your work is very confidential and you can't publicly discuss it, you can |
| 360 | also mail me directly about it, but your mail may be waiting several days in |
| 361 | the queue before you get a response. |
| 362 | |
| 363 | If you'd like a feature to be added but you think you don't have the skills to |
| 364 | implement it yourself, you should follow these steps : |
| 365 | |
| 366 | 1. discuss the feature on the mailing list. It is possible that someone |
| 367 | else has already implemented it, or that someone will tell you how to |
| 368 | proceed without it, or even why not to do it. It is also possible that |
| 369 | in fact it's quite easy to implement and people will guide you through |
| 370 | the process. That way you'll finally have YOUR patch merged, providing |
| 371 | the feature YOU need. |
| 372 | |
| 373 | 2. if you really can't code it yourself after discussing it, then you may |
| 374 | consider contacting someone to do the job for you. Some people on the |
Willy Tarreau | 869f351 | 2014-06-19 15:26:32 +0200 | [diff] [blame] | 375 | list might sometimes be OK with trying to do it. |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 376 | |
| 377 | Note to contributors: it's very handy when patches comes with a properly |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 378 | formated subject. There are 3 criteria of particular importance in any patch : |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 379 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 380 | - its nature (is it a fix for a bug, a new feature, an optimization, ...) |
| 381 | - its importance, which generally reflects the risk of merging/not merging it |
| 382 | - what area it applies to (eg: http, stats, startup, config, doc, ...) |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 383 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 384 | It's important to make these 3 criteria easy to spot in the patch's subject, |
| 385 | because it's the first (and sometimes the only) thing which is read when |
| 386 | reviewing patches to find which ones need to be backported to older versions. |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 387 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 388 | Specifically, bugs must be clearly easy to spot so that they're never missed. |
| 389 | Any patch fixing a bug must have the "BUG" tag in its subject. Most common |
| 390 | patch types include : |
| 391 | |
| 392 | - BUG fix for a bug. The severity of the bug should also be indicated |
| 393 | when known. Similarly, if a backport is needed to older versions, |
| 394 | it should be indicated on the last line of the commit message. If |
| 395 | the bug has been identified as a regression brought by a specific |
| 396 | patch or version, this indication will be appreciated too. New |
| 397 | maintenance releases are generally emitted when a few of these |
| 398 | patches are merged. |
| 399 | |
| 400 | - CLEANUP code cleanup, silence of warnings, etc... theorically no impact. |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 401 | These patches will rarely be seen in stable branches, though they |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 402 | may appear when they remove some annoyance or when they make |
| 403 | backporting easier. By nature, a cleanup is always minor. |
| 404 | |
| 405 | - REORG code reorganization. Some blocks may be moved to other places, |
| 406 | some important checks might be swapped, etc... These changes |
| 407 | always present a risk of regression. For this reason, they should |
| 408 | never be mixed with any bug fix nor functional change. Code is |
| 409 | only moved as-is. Indicating the risk of breakage is highly |
| 410 | recommended. |
| 411 | |
| 412 | - BUILD updates or fixes for build issues. Changes to makefiles also fall |
| 413 | into this category. The risk of breakage should be indicated if |
| 414 | known. It is also appreciated to indicate what platforms and/or |
| 415 | configurations were tested after the change. |
| 416 | |
| 417 | - OPTIM some code was optimised. Sometimes if the regression risk is very |
| 418 | low and the gains significant, such patches may be merged in the |
| 419 | stable branch. Depending on the amount of code changed or replaced |
| 420 | and the level of trust the author has in the change, the risk of |
| 421 | regression should be indicated. |
| 422 | |
| 423 | - RELEASE release of a new version (development or stable). |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 424 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 425 | - LICENSE licensing updates (may impact distro packagers). |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 426 | |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 427 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 428 | When the patch cannot be categorized, it's best not to put any tag. This is |
| 429 | commonly the case for new features, which development versions are mostly made |
| 430 | of. |
| 431 | |
| 432 | Additionally, the importance of the patch should be indicated when known. A |
| 433 | single upper-case word is preferred, among : |
| 434 | |
| 435 | - MINOR minor change, very low risk of impact. It is often the case for |
| 436 | code additions that don't touch live code. For a bug, it generally |
| 437 | indicates an annoyance, nothing more. |
| 438 | |
| 439 | - MEDIUM medium risk, may cause unexpected regressions of low importance or |
| 440 | which may quickly be discovered. For a bug, it generally indicates |
| 441 | something odd which requires changing the configuration in an |
| 442 | undesired way to work around the issue. |
| 443 | |
| 444 | - MAJOR major risk of hidden regression. This happens when I rearrange |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 445 | large parts of code, when I play with timeouts, with variable |
| 446 | initializations, etc... We should only exceptionally find such |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 447 | patches in stable branches. For a bug, it indicates severe |
| 448 | reliability issues for which workarounds are identified with or |
| 449 | without performance impacts. |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 450 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 451 | - CRITICAL medium-term reliability or security is at risk and workarounds, |
| 452 | if they exist, might not always be acceptable. An upgrade is |
| 453 | absolutely required. A maintenance release may be emitted even if |
| 454 | only one of these bugs are fixed. Note that this tag is only used |
| 455 | with bugs. Such patches must indicate what is the first version |
| 456 | affected, and if known, the commit ID which introduced the issue. |
| 457 | |
| 458 | If this criterion doesn't apply, it's best not to put it. For instance, most |
| 459 | doc updates and most examples or test files are just added or updated without |
| 460 | any need to qualify a level of importance. |
| 461 | |
| 462 | The area the patch applies to is quite important, because some areas are known |
| 463 | to be similar in older versions, suggesting a backport might be desirable, and |
| 464 | conversely, some areas are known to be specific to one version. When the tag is |
| 465 | used alone, uppercase is preferred for readability, otherwise lowercase is fine |
| 466 | too. The following tags are suggested but not limitative : |
| 467 | |
| 468 | - doc documentation updates or fixes. No code is affected, no need to |
| 469 | upgrade. These patches can also be sent right after a new feature, |
| 470 | to document it. |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 471 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 472 | - examples example files. Be careful, sometimes these files are packaged. |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 473 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 474 | - tests regression test files. No code is affected, no need to upgrade. |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 475 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 476 | - init initialization code, arguments parsing, etc... |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 477 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 478 | - config configuration parser, mostly used when adding new config keywords |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 479 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 480 | - http the HTTP engine |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 481 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 482 | - stats the stats reporting engine as well as the stats socket CLI |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 483 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 484 | - checks the health checks engine (eg: when adding new checks) |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 485 | |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 486 | - acl the ACL processing core or some ACLs from other areas |
| 487 | |
| 488 | - peers the peer synchronization engine |
| 489 | |
| 490 | - listeners everything related to incoming connection settings |
| 491 | |
| 492 | - frontend everything related to incoming connection processing |
| 493 | |
| 494 | - backend everything related to LB algorithms and server farm |
| 495 | |
| 496 | - session session processing and flags (very sensible, be careful) |
| 497 | |
| 498 | - server server connection management, queueing |
| 499 | |
| 500 | - proxy proxy maintenance (start/stop) |
| 501 | |
| 502 | - log log management |
| 503 | |
| 504 | - poll any of the pollers |
| 505 | |
| 506 | - halog the halog sub-component in the contrib directory |
| 507 | |
| 508 | - contrib any addition to the contrib directory |
| 509 | |
| 510 | Other names may be invented when more precise indications are meaningful, for |
| 511 | instance : "cookie" which indicates cookie processing in the HTTP core. Last, |
| 512 | indicating the name of the affected file is also a good way to quickly spot |
| 513 | changes. Many commits were already tagged with "stream_sock" or "cfgparse" for |
| 514 | instance. |
| 515 | |
| 516 | It is desired that AT LEAST one of the 3 criteria tags is reported in the patch |
| 517 | subject. Ideally, we would have the 3 most often. The two first criteria should |
| 518 | be present before a first colon (':'). If both are present, then they should be |
| 519 | delimited with a slash ('/'). The 3rd criterion (area) should appear next, also |
| 520 | followed by a colon. Thus, all of the following messages are valid : |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 521 | |
| 522 | Examples of messages : |
Willy Tarreau | 9a639a1 | 2011-09-10 22:48:36 +0200 | [diff] [blame] | 523 | - DOC: document options forwardfor to logasap |
| 524 | - DOC/MAJOR: reorganize the whole document and change indenting |
| 525 | - BUG: stats: connection reset counters must be plain ascii, not HTML |
| 526 | - BUG/MINOR: stats: connection reset counters must be plain ascii, not HTML |
| 527 | - MEDIUM: checks: support multi-packet health check responses |
| 528 | - RELEASE: Released version 1.4.2 |
| 529 | - BUILD: stats: stdint is not present on solaris |
| 530 | - OPTIM/MINOR: halog: make fgets parse more bytes by blocks |
| 531 | - REORG/MEDIUM: move syscall redefinition to specific places |
| 532 | |
| 533 | Please do not use square brackets anymore around the tags, because they give me |
| 534 | more work when merging patches. By default I'm asking Git to keep them but this |
| 535 | causes trouble when patches are prefixed with the [PATCH] tag because in order |
| 536 | not to store it, I have to hand-edit the patches. So as of now, I will ask Git |
| 537 | to remove whatever is located between square brackets, which implies that any |
| 538 | subject formatted the old way will have its tag stripped out. |
| 539 | |
| 540 | In fact, one of the only square bracket tags that still makes sense is '[RFC]' |
| 541 | at the beginning of the subject, when you're asking for someone to review your |
| 542 | change before getting it merged. If the patch is OK to be merged, then I can |
| 543 | merge it as-is and the '[RFC]' tag will automatically be removed. If you don't |
| 544 | want it to be merged at all, you can simply state it in the message, or use an |
| 545 | alternate '[WIP]' tag ("work in progress"). |
| 546 | |
| 547 | The tags are not rigid, follow your intuition first, anyway I reserve the right |
| 548 | to change them when merging the patch. It may happen that a same patch has a |
| 549 | different tag in two distinct branches. The reason is that a bug in one branch |
| 550 | may just be a cleanup in the other one because the code cannot be triggered. |
| 551 | |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 552 | |
| 553 | For a more efficient interaction between the mainline code and your code, I can |
| 554 | only strongly encourage you to try the Git version control system : |
| 555 | |
| 556 | http://git-scm.com/ |
| 557 | |
| 558 | It's very fast, lightweight and lets you undo/redo your work as often as you |
| 559 | want, without making your mistakes visible to the rest of the world. It will |
| 560 | definitely help you contribute quality code and take other people's feedback |
| 561 | in consideration. In order to clone the HAProxy Git repository : |
| 562 | |
Willy Tarreau | 869f351 | 2014-06-19 15:26:32 +0200 | [diff] [blame] | 563 | $ git clone http://git.haproxy.org/git/haproxy-1.5.git (stable 1.5) |
Willy Tarreau | 6346f0a | 2014-05-10 11:04:39 +0200 | [diff] [blame] | 564 | $ git clone http://git.haproxy.org/git/haproxy.git/ (development) |
Willy Tarreau | 663148c | 2012-12-12 00:38:22 +0100 | [diff] [blame] | 565 | |
Willy Tarreau | b1a34b6 | 2010-05-09 22:37:12 +0200 | [diff] [blame] | 566 | If you decide to use Git for your developments, then your commit messages will |
| 567 | have the subject line in the format described above, then the whole description |
| 568 | of your work (mainly why you did it) will be in the body. You can directly send |
| 569 | your commits to the mailing list, the format is convenient to read and process. |
| 570 | |
willy tarreau | 7834533 | 2005-12-18 01:33:16 +0100 | [diff] [blame] | 571 | -- end |