Skip to content

Commit

Permalink
CI(report-benchmarks-failures): fix condition (#7745)
Browse files Browse the repository at this point in the history
## Problem

`report-benchmarks-failures` job is triggered for any failure in the CI
pipeline, but we need it to be triggered only for failed `benchmarks`
job

## Summary of changes
- replace `failure()` with `needs.benchmarks.result == 'failure'` in the
condition
  • Loading branch information
bayandin committed May 14, 2024
1 parent df0f1e3 commit b6ee918
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/build_and_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -548,7 +548,7 @@ jobs:

report-benchmarks-failures:
needs: [ benchmarks, create-test-report ]
if: github.ref_name == 'main' && failure()
if: github.ref_name == 'main' && needs.benchmarks.result == 'failure'
runs-on: ubuntu-latest

steps:
Expand Down

1 comment on commit b6ee918

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3141 tests run: 3001 passed, 0 failed, 140 skipped (full report)


Flaky tests (1)

Postgres 16

  • test_vm_bit_clear_on_heap_lock: debug

Code coverage* (full report)

  • functions: 31.4% (6338 of 20189 functions)
  • lines: 47.4% (47946 of 101250 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
b6ee918 at 2024-05-14T12:06:20.723Z :recycle:

Please sign in to comment.