Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable fail-fast for GitHub Actions Jobs with a Matrix Strategy #88

Merged

Conversation

mcdonnnj
Copy link
Member

🗣 Description

This pr sets fail-fast: false for any GitHub Actions job that employs a matrix strategy.

💭 Motivation and context

This ensures that all generated runs for the matrix strategy run to completion regardless of the finishing state of other runs.

🧪 Testing

Automated tests pass. The same setting is used for our Ansible role test job.

✅ Checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • Tests have been added and/or modified to cover the changes in this PR.
  • All new and existing tests pass.

Override the default by setting `fail-fast` to `false` for the test and build
jobs in our GHA build workflow. This ensure that all matrix jobs run regardless
of a failure.
@mcdonnnj mcdonnnj added the improvement This issue or pull request will add or improve functionality, maintainability, or ease of use label Oct 29, 2021
@mcdonnnj mcdonnnj requested a review from dav3r as a code owner October 29, 2021 19:18
@mcdonnnj mcdonnnj self-assigned this Oct 29, 2021
Copy link
Member

@dav3r dav3r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦊 🦊 🦊 🦊 🦊

Copy link
Member

@felddy felddy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fail fast, fail continuously! ⏩ 💀
💪

@mcdonnnj mcdonnnj merged commit 0a70bbb into develop Oct 29, 2021
@mcdonnnj mcdonnnj deleted the improvement/disable_fail-fast_for_test_and_build_jobs branch October 29, 2021 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement This issue or pull request will add or improve functionality, maintainability, or ease of use
Projects
Development

Successfully merging this pull request may close these issues.

4 participants