Skip to content

Commit 5b937e9

Browse files
authored
ci: add system-tests finished job (#10699) (#10880)
## Checklist - [x] PR author has checked that all the criteria below are met - The PR description includes an overview of the change - The PR description articulates the motivation for the change - The change includes tests OR the PR description describes a testing strategy - The PR description notes risks associated with the change, if any - Newly-added code is easy to change - The change follows the [library release note guidelines](https://ddtrace.readthedocs.io/en/stable/releasenotes.html) - The change includes or references documentation updates if necessary - Backport labels are set (if [applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)) ## Reviewer Checklist - [x] Reviewer has checked that all the criteria below are met - Title is accurate - All changes are related to the pull request's stated goal - Avoids breaking [API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces) changes - Testing strategy adequately addresses listed risks - Newly-added code is easy to change - Release note makes sense to a user of the library - If necessary, author has acknowledged and discussed the performance implications of this PR as reported in the benchmarks PR comment - Backport labels are set in a manner that is consistent with the [release branch maintenance policy](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)
1 parent 0911fa1 commit 5b937e9

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

.github/workflows/system-tests.yml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -300,3 +300,14 @@ jobs:
300300
name: logs_parametric
301301
path: artifact.tar.gz
302302

303+
finished:
304+
runs-on: ubuntu-latest
305+
needs: [parametric, system-tests]
306+
if: success() || failure()
307+
steps:
308+
- name: True when everything else succeeded
309+
if: needs.parametric.result == 'success' && needs.system-tests.result == 'success'
310+
run: exit 0
311+
- name: Fails if anything else failed
312+
if: needs.parametric.result != 'success' || needs.system-tests.result != 'success'
313+
run: exit 1

0 commit comments

Comments
 (0)