On 10/04/17 11:38, [email protected] wrote:
I test also 1.7.9 debian backported and it still fails.

Is my configuration so out of this world? I would prefer not to have a sepereate listener for every port I route through haproxy.

Hi,

I tested last 1.8.dev3 today and my error with multiple ports beeing in tcp-check is not there anymore.

I bisected the code and found the commit, that fixed my problem:

f411cce456dabbe3cf6637f2bb11435af0499ae7 is the first fixed commit
commit f411cce456dabbe3cf6637f2bb11435af0499ae7
Author: Willy Tarreau <[email protected]>
Date:   Wed Oct 4 16:21:19 2017 +0200

    MINOR: checks: don't create then kill a dummy connection before tcp-checks

    When tcp-checks are in use, a connection starts to be created, then it's
    destroyed so that tcp-check can recreate its own. Now we directly move
    to tcpcheck_main() when it's detected that tcp-check is in use.

:040000 040000 a4695d29907baeac642ae68e1f7d8776d8b93aea 66a28a0fb9257f20e8513d5c28ca99cb0bd66b72 M      src




@Willy: thanks for taking a look at this issue.

Is it possible to backport the changes to tcp-check to 1.7?


Additionally I added a multi-port, multi-host listener to the tcp-check test config. See attached patch file.


thanks
Philipp

--
-----------------------------------------------------------------------
DI Mag. Philipp Kolmann              mail: [email protected]
Technische Universitaet Wien                  web: www.zid.tuwien.ac.at
Zentraler Informatikdienst (ZID)                 tel: +43(1)58801-42011
Wiedner Hauptstr. 8-10, A-1040 Wien                        DVR: 0005886
-----------------------------------------------------------------------

>From 944610de46bb3b23d514a12699690fd2acffc2a2 Mon Sep 17 00:00:00 2001
From: Philipp Kolmann <[email protected]>
Date: Tue, 24 Oct 2017 11:32:22 +0200
Subject: [PATCH] Add a testcase for my multi-port + multi-server listener
 issue

---
 tests/tcp-check.cfg | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/tests/tcp-check.cfg b/tests/tcp-check.cfg
index 16d2836e..56aba53c 100644
--- a/tests/tcp-check.cfg
+++ b/tests/tcp-check.cfg
@@ -13,6 +13,17 @@ listen stats
 	bind :8888
 	stats uri /
 
+listen multi-tcp-check
+        bind :80
+        mode tcp
+        option tcp-check
+        tcp-check connect port 9001
+        tcp-check connect port 9002
+
+        server server1 127.0.0.1: check
+        server server2 127.0.0.2: check
+        server server3 127.0.0.3: check
+
 backend tcp9000
 	option tcp-check
 	server srv 127.0.0.1:9000 check inter 1s
-- 
2.11.0

Reply via email to