[RATIS-1341] Fail-fast tests only for PRs

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.1.0
• Component/s: build
• Target Version/s: 2.1.0
• Assignee: Attila Doroszlai
• Reporter: Attila Doroszlai
• Created: 17/Mar/21 11:54
• Updated: 18/Mar/21 14:22
• Resolved: 18/Mar/21 13:57

Description

RATIS-1313 added matrix build for unit tests with 3 splits, with fail-fast behavior (if one split fails, other splits are cancelled). szetszwo pointed out that

better to always run all the tests since the overall running time is short. This is the reason that we moved the tests to the ratis-test module; see RATIS-399

I propose to run all tests for push builds, but keep fail-fast behavior for PRs. The rationale is that any failing test requires a new run, because PRs should be merged only after fully green build.

Issue Links

• is related to: RATIS-1313 Use matrix build for unit tests
• links to: GitHub Pull Request #450