2 Commits (264bb374b950d1811d81c4f1f6d59352da2ec372)

Author	SHA1	Message	Date
James Cammarata	ca5b361ad8	Reworking iterator logic regarding failed states during always ... Previous changes addressed a corner case, which unfortunately introduced another bug. This patch adds a new flag to the host state (did_rescue) which is set to true when the rescue portion of a block completes. This flag is then checked in _check_failed_state() when the fail_state != FAILED_NONE. This lead to the discovery of another bug - current strategies are not advancing hosts to ITERATING_COMPLETE after doing a peek at the next task, leaving the host state in the run_state of the final task. To address this, before gathering the list of failed hosts in StrategyBase.run(), a final pass through the iterator for all hosts is done to ensure each host is in its final state. This way, no strategy derived from StrategyBase has to worry about it and it's handled. Fixes #17983	8 years ago
Matt Clay	ac8842eee8	Fix unit test dirs to match code under test.	8 years ago