f4bd39d2de
GitHub actions' YAML supports the `>` directive to a multi-line command to be merged into a single line, but requires that all segments have no leading whitespace otherwise they get silently ignored. Since this is not nearly as obvious as one might expect, fallback to just the `|` syntax using `\` to extend the lines and keep the overall formatting, fixing the reporting of retried tests via curl webhook. |
||
|---|---|---|
| .. | ||
| api.yml | ||
| artifacts.yml | ||
| codeql-analysis.yml | ||
| docker-push-mirrored.yml | ||
| e2e-tests-ci.yml | ||
| esrupgrade-common.yml | ||
| esrupgrade.yml | ||
| mmctl-test-template.yml | ||
| README.md | ||
| scorecards-analysis.yml | ||
| server-ci-master.yml | ||
| server-ci-pr.yml | ||
| server-ci-template.yml | ||
| server-test-template.yml | ||
| webapp-ci-master.yml | ||
| webapp-ci-pr.yml | ||
| webapp-ci-template.yml | ||
Background
This document aims to explain the bunch of server and webapp yaml files and their functionality.
The context behind this complexity is that we want new pushes to PR branches to cancel older in-progress and pending CI runs, but we don't want that to happen in master branch. Unfortunately, there is no config knob to control pending workflows and if you set a concurrency group, then pending workflows will always be canceled. Refer to https://github.com/orgs/community/discussions/5435 for discussion.
Therefore, we have a template yaml file which is actually the main CI code. That is then imported by {server|webapp}-ci-master.yml and {server|webapp}-ci-pr.yml. The -master.yml files don't have any concurrency limits, but -pr.yml files do.
Folder structure
server-ci-pr | ---server-ci-template | ---server-test-template (common code for postgres and mysql tests)
server-ci-master | ---server-ci-template | ---server-test-template (common code for postgres and mysql tests)
webapp-ci-pr | ---webapp-ci-template
webapp-ci-master | ---webapp-ci-template