Skip to content

Commit 01333bd

Browse files
committed
Another attempt to mark the job as yellow if warnings/errors increased
1 parent 6ac1696 commit 01333bd

2 files changed

Lines changed: 14 additions & 12 deletions

File tree

.github/actions/full-build-metrics/action.yml

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,11 @@ inputs:
2323
required: false
2424
default: "90"
2525

26+
outputs:
27+
regression:
28+
description: Whether warnings/errors increased versus baseline
29+
value: ${{ steps.count.outputs.regression }}
30+
2631
runs:
2732
using: composite
2833
steps:
@@ -37,6 +42,7 @@ runs:
3742
if_no_artifact_found: warn
3843

3944
- name: Count warnings/errors and compare
45+
id: count
4046
shell: bash
4147
run: |
4248
if [ -f "${{ inputs.log-file }}" ]; then
@@ -89,11 +95,7 @@ runs:
8995
fi
9096
} >> "$GITHUB_STEP_SUMMARY"
9197
92-
if [ "${regression}" = "true" ]; then
93-
echo "METRICS_REGRESSION=true" >> "$GITHUB_ENV"
94-
else
95-
echo "METRICS_REGRESSION=false" >> "$GITHUB_ENV"
96-
fi
98+
echo "regression=${regression}" >> "$GITHUB_OUTPUT"
9799
98100
- name: Upload current metrics artifact
99101
uses: actions/upload-artifact@v7
@@ -103,9 +105,3 @@ runs:
103105
retention-days: ${{ inputs.retention-days }}
104106
if-no-files-found: error
105107

106-
- name: Mark metrics regression
107-
shell: bash
108-
run: |
109-
if [ "${METRICS_REGRESSION:-false}" = "true" ]; then
110-
exit 1
111-
fi

.github/workflows/ci.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ jobs:
164164
name: full-build-base-log-current
165165
path: .
166166
- name: Analyze and persist full-build-base metrics
167-
continue-on-error: true
167+
id: metrics
168168
uses: ./.github/actions/full-build-metrics
169169
with:
170170
log-file: full-build-base.log
@@ -173,6 +173,12 @@ jobs:
173173
workflow-file: ci.yml
174174
#These metrics occupy very little space, increasing the retention so they are not lost if nothing is merged to master in a while
175175
retention-days: 90
176+
- name: Mark metrics regression
177+
if: steps.metrics.outputs.regression == 'true'
178+
continue-on-error: true
179+
run: |
180+
echo "::warning::Warnings/errors increased versus baseline."
181+
exit 1
176182
177183
base-build-mac:
178184
runs-on: macos-latest

0 commit comments

Comments
 (0)