Skip to content

Commit b33211d

Browse files
committed
fix: wait for medium e2e to finish and pass
Signed-off-by: Charlie Doern <[email protected]>
1 parent aacd5b4 commit b33211d

File tree

3 files changed

+88
-1
lines changed

3 files changed

+88
-1
lines changed

.github/mergify.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,22 @@ pull_request_rules:
2424
- -files~=^\.github/(actions|workflows)/.*\.ya?ml$
2525
- -files~=^\.github/workflows/actionlint\.
2626

27+
# medium e2e workflow
28+
- or:
29+
- and:
30+
# note this should match the triggering criteria in 'e2e-nvidia-l4-x1.yml'
31+
- check-success~=e2e-medium-workflow-complete
32+
- or:
33+
- files~=\.py$
34+
- files=pyproject.toml
35+
- files~=^requirements.*\.txt$
36+
- files~=^scripts/[^/]+\.sh$
37+
- and:
38+
- -files~=\.py$
39+
- -files=pyproject.toml
40+
- -files~=^requirements.*\.txt$
41+
- -files~=^scripts/[^/]+\.sh$
42+
2743
# code lint workflow
2844
- or:
2945
- and:

.github/workflows/e2e-nvidia-l40s-x4-sdk.yml

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -324,4 +324,15 @@ jobs:
324324
if: steps.phase-2-upload-s3.outcome == 'failure'
325325
run: |
326326
echo "::warning::Failed to upload Phase 2 loss graph to S3. This won't block the workflow, but you may want to investigate."
327-
echo "Loss graph upload failed" >> "${GITHUB_STEP_SUMMARY}"
327+
echo "Loss graph upload failed" >> "${GITHUB_STEP_SUMMARY}"
328+
329+
e2e-medium-workflow-complete:
330+
# we don't want to block PRs on failed EC2 cleanup
331+
# so not requiring "stop-medium-ec2-runner" as well
332+
permissions:
333+
checks: read
334+
uses: ./.github/workflows/status-checks.yml
335+
with:
336+
job_ids: >- # Space-separated job ids to wait on for status checks
337+
start-large-ec2-runner
338+
e2e-medium-test
Lines changed: 60 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,60 @@
1+
# SPDX-License-Identifier: Apache-2.0
2+
# yamllint disable rule:line-length
3+
4+
name: Status Checks Reusable Workflow
5+
6+
on:
7+
workflow_call:
8+
inputs:
9+
job_ids:
10+
description: 'Space-separated job ids to wait on for status checks'
11+
required: true
12+
type: string
13+
delay:
14+
description: 'Period in seconds to wait before first poll of GitHub Check Runs'
15+
required: false
16+
type: number
17+
default: 10
18+
interval:
19+
description: 'Interval or period in seconds between polling GitHub Check Runs'
20+
required: false
21+
type: number
22+
default: 10
23+
timeout:
24+
description: 'Timeout in seconds to complete polling GitHub Check Runs'
25+
required: false
26+
type: number
27+
default: 3600
28+
29+
env:
30+
LC_ALL: en_US.UTF-8
31+
32+
defaults:
33+
run:
34+
shell: bash
35+
36+
permissions:
37+
checks: read
38+
39+
jobs:
40+
status-checks:
41+
runs-on: ubuntu-latest
42+
steps:
43+
- name: "Harden Runner"
44+
uses: step-security/harden-runner@0634a2670c59f64b4a01f0f96f84700a4088b9f0 # v2.12.0
45+
with:
46+
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs
47+
48+
- name: "Set status check variables"
49+
id: set_variables
50+
run: |
51+
jq -nr '[$ARGS.positional[] | split("\\s"; null) | map(select(. != ""))] | flatten | join("|") | ("match_pattern=(" + . + ")")' --args "${{ inputs.job_ids }}" >> "$GITHUB_OUTPUT"
52+
53+
- name: "Wait for status checks"
54+
uses: poseidon/wait-for-status-checks@899c768d191b56eef585c18f8558da19e1f3e707 # v0.6.0
55+
with:
56+
token: ${{ secrets.GITHUB_TOKEN }}
57+
match_pattern: ${{ steps.set_variables.outputs.match_pattern }}
58+
delay: ${{ inputs.delay }}
59+
interval: ${{ inputs.interval }}
60+
timeout: ${{ inputs.timeout }}

0 commit comments

Comments
 (0)