Skip to content

Commit e52d587

Browse files
committed
chore: also upload test run error logs
1 parent 2b7b423 commit e52d587

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

.github/workflows/configure-rules-for-test.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ while read -r line; do
2222
test_value="$(cut -d'|' -f5 <<< "${line}")"
2323

2424
if ! grep -q "${var_name}" "${readme_file}"; then
25-
echo "Failed to find environment variable `${var_name}` in README."
25+
echo "Failed to find environment variable '${var_name}' in README."
2626
exit 1
2727
fi
2828

.github/workflows/verifyimage.yml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,3 +147,12 @@ jobs:
147147
--log-file "tests/logs/${modsec_version}/error.log" \
148148
--overrides "tests/regression/${backend}-overrides.yaml" \
149149
--show-failures-only
150+
151+
- name: Upload logs
152+
uses: actions/upload-artifact@v4
153+
if: always()
154+
with:
155+
name: ${{ matrix.target }}-error.log
156+
path: "crs/tests/logs/${{ contains(matrix.target, 'apache') && 'modsec2-apache' || 'modsec3-nginx' }}/error.log"
157+
retention-days: 7
158+
overwrite: true

0 commit comments

Comments
 (0)