When porting variables from macros to dynamic config variables, we've lost coverage in complexity tests. An example is the EVENTS_MAP_RATE_LIMIT:
- in v1.19, we were running complexity tests with
-DEVENTS_MAP_RATE_LIMIT=1000
- in v1.20 we ported it to a
CONFIG variable, without adding it to the test permutations (or removing it afterwards to stay in the limit, haven't double checked). This means the respective codepath (i.e., if (CONFIG(events_map_rate_limit) > 0)) is never verified. (occurred on this backport)
This means we're potentially hitting complexity issues we're not aware of in main.
When porting variables from macros to dynamic config variables, we've lost coverage in complexity tests. An example is the
EVENTS_MAP_RATE_LIMIT:-DEVENTS_MAP_RATE_LIMIT=1000CONFIGvariable, without adding it to the test permutations (or removing it afterwards to stay in the limit, haven't double checked). This means the respective codepath (i.e.,if (CONFIG(events_map_rate_limit) > 0)) is never verified. (occurred on this backport)This means we're potentially hitting complexity issues we're not aware of in main.