Skip to content

Commit edac0ac

Browse files
committed
fix: correct container socket path in Dockerfile and release smoke test
The armor user owns /var/run/armor/ but the ENTRYPOINT was writing the socket to /var/run/armor.sock (parent directory, not writable). Changed to /var/run/armor/armor.sock throughout: ENTRYPOINT, HEALTHCHECK, and the release workflow smoke-test health check.
1 parent 57cc7d1 commit edac0ac

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ jobs:
166166
- name: Smoke test daemon health inside container
167167
run: |
168168
for i in {1..30}; do
169-
if docker exec armor-smoke armor health; then
169+
if docker exec armor-smoke armor health --socket /var/run/armor/armor.sock; then
170170
exit 0
171171
fi
172172
sleep 2

docker/Dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ USER armor
6262

6363
# Health check: verify daemon can start (this will fail initially until daemon starts listening)
6464
HEALTHCHECK --interval=30s --timeout=5s --start-period=10s --retries=3 \
65-
CMD test -S /var/run/armor.sock || exit 1
65+
CMD test -S /var/run/armor/armor.sock || exit 1
6666

6767
# Entry point: run the daemon with the baked model
68-
ENTRYPOINT ["armor", "daemon", "--socket", "/var/run/armor.sock", "--db", "/var/lib/armor/armor.db", "--model", "/models/active.gguf"]
68+
ENTRYPOINT ["armor", "daemon", "--socket", "/var/run/armor/armor.sock", "--db", "/var/lib/armor/armor.db", "--model", "/models/active.gguf"]

0 commit comments

Comments
 (0)