commit 107ddc1b67ad43b1406e49ea1d45010755dd60e5
author Adam Spiers <aspiers@suse.com> Mon Feb 23 15:28:41 2015 +0000
committer Willy Tarreau <w@1wt.eu> Mon Feb 23 18:13:14 2015 +0100
tree 3c5254148eded0331d1cae9d1366a6697a056111
parent ecb74a3d86b1d89b5821667691b1a00ad5b21e4b

BUG/MINOR: don't start haproxy on reload

If haproxy is not already running, reload should not start it.
Unfortunately the LSB spec does not explicitly cover this case:

  http://refspecs.linuxbase.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generic/iniscrptact.html

but it seems like the more correct behaviour, and actually fixes a
real-world issue we saw in SUSE Cloud.

Signed-off-by: Adam Spiers <aspiers@suse.com>

examples/haproxy.init

diff --git a/examples/haproxy.init b/examples/haproxy.init
index bf13b15..d02c367 100644
--- a/examples/haproxy.init
+++ b/examples/haproxy.init
@@ -78,6 +78,10 @@
 }
 
 reload() {
+  if ! [ -s $PIDFILE ]; then
+    return 0
+  fi
+
   quiet_check
   if [ $? -ne 0 ]; then
     echo "Errors found in configuration file, check it with '$BASENAME check'."