Correct code comment in PBE to reflect current behavior (#83269)

Correct a code comment in the PlaybookExecutor class that explains
the host failure condition under which the PBE breaks out of the
run. This comment previously stated that the maximum failure
percentage was checked, but that logic has been refactored into
the linear strategy plugin.
pull/83295/head
Eric Appelt 6 months ago committed by GitHub
parent 560d3af097
commit 10f9b8e655
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

@ -195,10 +195,7 @@ class PlaybookExecutor:
result = self._tqm.RUN_FAILED_HOSTS result = self._tqm.RUN_FAILED_HOSTS
break_play = True break_play = True
# check the number of failures here, to see if they're above the maximum # check the number of failures here and break out if the entire batch failed
# failure percentage allowed, or if any errors are fatal. If either of those
# conditions are met, we break out, otherwise we only break out if the entire
# batch failed
failed_hosts_count = len(self._tqm._failed_hosts) + len(self._tqm._unreachable_hosts) - \ failed_hosts_count = len(self._tqm._failed_hosts) + len(self._tqm._unreachable_hosts) - \
(previously_failed + previously_unreachable) (previously_failed + previously_unreachable)

Loading…
Cancel
Save