Skip to content

Commit c36950d

Browse files
authored
Merge pull request containerd#9831 from mxpv/status
Workaround "required job statuses" job list
2 parents d346aee + a890c8e commit c36950d

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

.github/workflows/ci.yml

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -601,3 +601,20 @@ jobs:
601601
path: |
602602
*-junit.xml
603603
*-gotest.json
604+
605+
# Currently Github actions UI supports no masks to mark matrix jobs as required to pass status checks.
606+
# This means that every time version of Go, containerd, or OS is changed, a corresponding job should
607+
# be added to the list of required checks. Which is not very convenient.
608+
# To workaround this, a special job is added to report statuses of all other jobs, with fixed title.
609+
# So it needs to be added to the list of required checks only once.
610+
#
611+
# See https://github.com/orgs/community/discussions/26822
612+
results:
613+
name: Report required job statuses
614+
runs-on: ubuntu-latest
615+
# List job dependencies which are required to pass status checks in order to be merged via merge queue.
616+
needs: [linters, project, protos, binaries, integration-linux, integration-windows, integration-vagrant, tests-cri-in-userns, tests-mac-os]
617+
if: ${{ always() }}
618+
steps:
619+
- run: exit 1
620+
if: ${{ contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') }}

0 commit comments

Comments
 (0)