Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
test: Only call Fail() once for all error logs
We've recently expanded the fail cases for when different logs are seen in the cilium-agent logs. However at the same time we also introduced a behaviour where for every single instance of the failure in the logs, we will call Fail(). In local CI runs with `--holdEnvironment` this can lead to a loop where the test is suspended for debugging, the developer performs ^Z and `fg`, then the same failure prints again, potentially many times over before the test can finally proceed / clean up. Tidy it up by detecting any failure of this sort once, then failing out. Note that the actual failure logs should already be visible earlier in the test output so this should not hide the failures from the developer. Signed-off-by: Joe Stringer <joe@cilium.io>
- Loading branch information