Revert "BUG/MEDIUM: connections: force connections cleanup on server changes"
authorWilliam Dauchy <w.dauchy@criteo.com>
Tue, 2 Jun 2020 14:03:58 +0000 (16:03 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Tue, 2 Jun 2020 17:09:50 +0000 (19:09 +0200)
commita31179b920929e30f2ec970bf3a3dc26d9ae50cd
treedc081a589d34a0a8268c691a3e4086981419513e
parent36e14bd3189a239a146994e924c0b2fd5f7b6b46
Revert "BUG/MEDIUM: connections: force connections cleanup on server changes"

As explained by Christopher on github issue #665:
In 2.2, srv->idle_conns and srv->safe_conns are thread-safe lists. But
not in 2.1. So the patch must be reverted or the lists must be changed
to use mt_list instead. The same must be done in 2.0, but the mt_list
does not exist on this version.

I choose to revert it as the original bug is truly revealed in v2.2
after commit 079cb9af22da6 ("MEDIUM: connections: Revamp the way idle
connections are killed")

this should resolve github issue #665

this reverts commit 7eab37b6819af685c647cf5a581e29fca2f3e079.
this reverts commit 3ad3306ec0bcb0cd4ca2b9ba134ed67663473ee8.

The patch was directly introduced on 2.1, there is no upstream commit ID for
this patch.
src/server.c