Skip to content

Commit 60f2816

Browse files
committed
chore: exclude nginx-alpine builds from test runs
nginx-alpine builds don't flush the log reliably. Refs coreruleset/go-ftw#473
1 parent e52d587 commit 60f2816

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

.github/workflows/verifyimage.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,8 @@ jobs:
128128
's#image: owasp/modsecurity-crs:${{ contains(matrix.target, 'apache') && 'apache' || 'nginx' }}.*#image: ${{ matrix.target }}-verification#' \
129129
crs/tests/docker-compose.yml
130130
- name: Run CRS tests for ${{ matrix.target }}
131+
# Log flushing isn't reliable enough for go-ftw (https://github.com/coreruleset/go-ftw/issues/473)
132+
if: ${{!contains(matrix.target, 'nginx-alpine')}}
131133
run: |
132134
cd crs
133135
modsec_version="${{ contains(matrix.target, 'apache') && 'modsec2-apache' || 'modsec3-nginx' }}"

0 commit comments

Comments
 (0)