Skip to content

Commit c48e39d

Browse files
authored
[CI] [debian] Containerize testing of standalone (#1924)
This should hopefully deal with #1394 (comment)
1 parent da18c98 commit c48e39d

File tree

1 file changed

+37
-9
lines changed

1 file changed

+37
-9
lines changed

.github/workflows/coq-debian.yml

+37-9
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ jobs:
102102
- name: display per-line timing info
103103
run: etc/ci/github-actions-display-per-line-timing.sh
104104

105-
test-standalone:
105+
test-standalone-host:
106106
strategy:
107107
fail-fast: false
108108
matrix:
@@ -135,10 +135,34 @@ jobs:
135135
printf '::warning::Debian ${{ matrix.debian }} binary does not run on ubuntu: %s\n' \
136136
"$(etc/ci/test-run-fiat-crypto.sh dist/fiat_crypto 2>&1 | tr '\n' '~' | sed 's/~/%0A/g')";
137137
}
138-
- name: setup Debian chroot
139-
run: etc/ci/setup-debian-chroot.sh "${{ matrix.debian }}"
138+
139+
test-standalone-container:
140+
strategy:
141+
fail-fast: false
142+
matrix:
143+
include:
144+
- debian: sid
145+
#- debian: bookworm # restore once 8.17 lands in Debian stable
146+
runs-on: ubuntu-latest
147+
container: debian:${{ matrix.debian }}
148+
needs: build
149+
steps:
150+
- uses: actions/checkout@v4
151+
- name: Download standalone ${{ matrix.debian }}
152+
uses: actions/download-artifact@v4
153+
with:
154+
name: standalone-${{ matrix.debian }}
155+
path: dist/
156+
- name: install system dependencies
157+
run: |
158+
apt-get -o Acquire::Retries=30 update -y
159+
apt-get -q -y --allow-unauthenticated -o Acquire::Retries=30 install sudo time file
160+
- name: List files
161+
run: find dist
162+
- run: chmod +x dist/fiat_crypto
163+
- name: host build params
164+
run: etc/ci/describe-system-config.sh
140165
- name: Test files (container)
141-
shell: in-debian-chroot.sh {0}
142166
run: |
143167
echo "::group::file fiat_crypto"
144168
file dist/fiat_crypto
@@ -187,18 +211,22 @@ jobs:
187211

188212
debian-check-all:
189213
runs-on: ubuntu-latest
190-
needs: [build, test-standalone, publish-standalone-dry-run]
214+
needs: [build, test-standalone-host, test-standalone-container, publish-standalone-dry-run]
191215
if: always()
192216
steps:
193217
- run: echo 'build passed'
194218
if: ${{ needs.build.result == 'success' }}
195-
- run: echo 'test-standalone passed'
196-
if: ${{ needs.test-standalone.result == 'success' }}
219+
- run: echo 'test-standalone-host passed'
220+
if: ${{ needs.test-standalone-host.result == 'success' }}
221+
- run: echo 'test-standalone-container passed'
222+
if: ${{ needs.test-standalone-container.result == 'success' }}
197223
- run: echo 'publish-standalone-dry-run passed'
198224
if: ${{ needs.publish-standalone-dry-run.result == 'success' }}
199225
- run: echo 'build failed' && false
200226
if: ${{ needs.build.result != 'success' }}
201-
- run: echo 'test-standalone failed' && false
202-
if: ${{ needs.test-standalone.result != 'success' }}
227+
- run: echo 'test-standalone-host failed' && false
228+
if: ${{ needs.test-standalone-host.result != 'success' }}
229+
- run: echo 'test-standalone-container failed' && false
230+
if: ${{ needs.test-standalone-container.result != 'success' }}
203231
- run: echo 'publish-standalone-dry-run failed' && false
204232
if: ${{ needs.publish-standalone-dry-run.result != 'success' }}

0 commit comments

Comments
 (0)