Skip to content

Commit

Permalink
Remove linux debug target, and change test results publisher to test-…
Browse files Browse the repository at this point in the history
…summary/action because it is faster. (youtube#4573)

b/372303096

Change to use a new test publisher because the existing publisher can
take over an hour to run when there are many test failures:
https://github.com/youtube/cobalt/actions/runs/12302321578/job/34336630966?pr=4569

The new publisher reports >2000 failures with annotations in 3 seconds:

https://github.com/youtube/cobalt/actions/runs/12472477285/job/34812061568?pr=4573

Also remove debug from linux build targets because it takes >1 hour to
complete.
  • Loading branch information
imindich authored Jan 2, 2025
1 parent 56d05dc commit 8e7db49
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 5 deletions.
13 changes: 9 additions & 4 deletions .github/actions/process_test_results/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,17 @@ runs:
name: ${{ inputs.test_results_key }}
path: results/

- name: Publish Test Report
- name: Test Summary action
continue-on-error: true
uses: mikepenz/action-junit-report@992d97d6eb2e5f3de985fbf9df6a04386874114d
uses: test-summary/action@31493c76ec9e7aa675f1585d3ed6f1da69269a86
with:
report_paths: results/*.xml
skip_annotations: true
paths: "results/*.xml"
output: test-report-summary.md
show: "fail, skip"

- name: Output test summary markdown to github
run: cat test-report-summary.md >> $GITHUB_STEP_SUMMARY
shell: bash

- name: Get Datadog CLI
id: download-dd-cli
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ jobs:
matrix:
platform: ${{ fromJson(needs.initialize.outputs.platforms) }}
include: ${{ fromJson(needs.initialize.outputs.includes) }}
config: [devel, debug, qa, gold]
config: [devel, qa, gold]
container: ${{ needs.docker-build-image.outputs.docker_tag }}
env:
TEST_ARTIFACTS_KEY: ${{ matrix.platform }}_${{ matrix.name }}_test_artifacts
Expand Down

0 comments on commit 8e7db49

Please sign in to comment.