Skip to content

Commit 45cd2a5

Browse files
fix: use docker run for strfry to provide config file
The strfry image requires a config file at /app/strfry.conf. Service containers can't mount files, so switch to docker run with a generated default config via `strfry export-config`. Also dump container logs on health check failure for easier debugging. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 9bef291 commit 45cd2a5

1 file changed

Lines changed: 10 additions & 5 deletions

File tree

.github/workflows/ci.yml

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -103,11 +103,6 @@ jobs:
103103
runs-on: ubuntu-latest
104104
permissions:
105105
contents: read
106-
services:
107-
strfry:
108-
image: ghcr.io/hoytech/strfry@sha256:7cf3b97c66a8c3087e16e5d31a3bb8eb92312f08dd969ff3dced9625242ac55f # latest
109-
ports:
110-
- 7777:7777
111106
steps:
112107
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4.3.1
113108
with:
@@ -122,6 +117,15 @@ jobs:
122117
with:
123118
tool: cargo-nextest
124119

120+
- name: Start strfry relay
121+
run: |
122+
IMAGE="ghcr.io/hoytech/strfry@sha256:7cf3b97c66a8c3087e16e5d31a3bb8eb92312f08dd969ff3dced9625242ac55f"
123+
docker pull "$IMAGE"
124+
docker run --rm "$IMAGE" export-config > /tmp/strfry.conf
125+
docker run -d --name strfry -p 7777:7777 \
126+
-v /tmp/strfry.conf:/app/strfry.conf:ro \
127+
"$IMAGE" relay
128+
125129
- name: Wait for strfry
126130
run: |
127131
for i in $(seq 1 30); do
@@ -134,6 +138,7 @@ jobs:
134138
sleep 1
135139
done
136140
echo "strfry failed to start"
141+
docker logs strfry
137142
exit 1
138143
139144
- name: Run integration tests

0 commit comments

Comments
 (0)