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 ba265fe..2b5cb4b 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;