So contrary to the principal of least astonishment, setting the default failure_limit to None sets it to 1 due to https://phabricator.wikimedia.org/source/scap/browse/master/scap/targets.py;2c4894c1a1ec4cfdad3c5981325267ec434f7852$273
There is probably a bit of refactoring that ought to happen to make the default failure_limit be unset (which does seem to make sense in hindsight :)). @dduvall is definitely the person I'd defer to for opinions here.
This patch was mainly about pointing out the inconsistency between the docs,
failure_limit [group]_failure_limit 1
and my experience,