git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] ci: do not cancel all jobs of a matrix if one fails

From
Jeff King <peff@peff.net>
Date
Jan 28, 2021, 06:35 UTC
Message-ID
<YBJbJOLyh+ksolVB@coredump.intra.peff.net>
In-Reply-To
<pull.851.git.1611806768911.gitgitgadget@gmail.com>
On Thu, Jan 28, 2021 at 04:06:08AM +0000, Philippe Blain via GitGitGadget wrote:
Show 13 quoted lines
> From: Philippe Blain <levraiphilippeblain@gmail.com>
> 
> The CI/PR GitHub Actions workflow uses the 'matrix' strategy for the
> "windows-test", "vs-test", "regular" and "dockerized" jobs. The default
> behaviour of GitHub Actions is to cancel all in-progress jobs in a
> matrix if one of the job of the matrix fails [1].
> 
> This is not ideal as a failure early in a job, like during installation of
> the build/test dependencies on a specific platform, leads to the
> cancellation of all other jobs in the matrix.
> 
> Set the 'fail-fast' variable to 'false' for all four matrix jobs in the
> workflow.
I think this is worth doing. Thanks for rolling it up into a patch.
-Peff
Previous: Philippe Blain via GitGitGadgetNext: Johannes Schindelin
Message 2 of 3 in “ci: do not cancel all jobs of a matrix if one fails”
  1. ci: do not cancel all jobs of a matrix if one failsPhilippe Blain via GitGitGadget, Jan 28, 2021
  2. Jeff KingJan 28, 2021
  3. Johannes SchindelinJan 28, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.