]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: backend: remove invalid mode test for "hash-balance-factor"
authorAurelien DARRAGON <adarragon@haproxy.com>
Mon, 20 Nov 2023 10:33:20 +0000 (11:33 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Fri, 24 Nov 2023 15:27:55 +0000 (16:27 +0100)
This is a leftover from 1e0093a317 ("MINOR: backend/balance: "balance"
requires TCP or HTTP mode").

Indeed, we cannot perform the test during parsing as the effective proxy
type is not yet known. Moreover, thanks to b61147fd ("MEDIUM: log/balance:
merge tcp/http algo with log ones") we could potentially benefit from
this setting even in log mode, but for now it is ignored by all log
compatible load-balancing algorithms.

src/cfgparse-listen.c

index ba265fe7606a0f7689f914b550d3ce2725c35d61..2b5cb4b53c5a54fceb7599735528c3f7e00f1f52 100644 (file)
@@ -2571,12 +2571,6 @@ stats_error_parsing:
                }
        }
        else if (strcmp(args[0], "hash-balance-factor") == 0) {
-               if (curproxy->mode != PR_MODE_TCP && curproxy->mode != PR_MODE_HTTP) {
-                       ha_alert("parsing [%s:%d] : '%s' requires TCP or HTTP mode.\n", file, linenum, args[0]);
-                       err_code |= ERR_ALERT | ERR_FATAL;
-                       goto out;
-               }
-
                if (*(args[1]) == 0) {
                        ha_alert("parsing [%s:%d] : '%s' expects an integer argument.\n", file, linenum, args[0]);
                        err_code |= ERR_ALERT | ERR_FATAL;