Skip to content

Commit 4f2f87d

Browse files
boomanaiden154github-actions[bot]
authored andcommitted
Automerge: [CI][Github] Fix Premerge Summary for Build Failures (#153076)
We were passing the wrong directory to `generate_test_report_github.py`, so no logs actually got passed there. This fixes that and ensures that the logs are uploaded as artifacts to enable future debugging.
2 parents 8c143be + fb70f51 commit 4f2f87d

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

.ci/utils.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,14 +26,15 @@ function at-exit {
2626
mkdir -p artifacts
2727
sccache --show-stats >> artifacts/sccache_stats.txt
2828
cp "${BUILD_DIR}"/.ninja_log artifacts/.ninja_log
29+
cp "${MONOREPO_ROOT}"/*.log artifacts/ || :
2930
cp "${BUILD_DIR}"/test-results.*.xml artifacts/ || :
3031

3132
# If building fails there will be no results files.
3233
shopt -s nullglob
3334

3435
if [[ "$GITHUB_STEP_SUMMARY" != "" ]]; then
3536
python "${MONOREPO_ROOT}"/.ci/generate_test_report_github.py \
36-
$retcode "${BUILD_DIR}"/test-results.*.xml "${BUILD_DIR}"/ninja*.log \
37+
$retcode "${BUILD_DIR}"/test-results.*.xml "${MONOREPO_ROOT}"/ninja*.log \
3738
>> $GITHUB_STEP_SUMMARY
3839
fi
3940
}

0 commit comments

Comments
 (0)