chatterbox: switch health probe from /health (doesn't exist) to /api/model-info
devnen/Chatterbox-TTS-Server doesn't expose /health — neither in code
nor OpenAPI. The deploy hung on the playbook's `Wait for /health to
respond` loop indefinitely (each curl -> 404, retry forever) even
though the container was up and the model loaded clean to CUDA at
22:52:21 (~42s after start).
/api/model-info returns `{"loaded":true,...}` only after the model
finishes loading, so it doubles as liveness + readiness. Updated:
* compose.yaml healthcheck — grep for `"loaded":true` from
/api/model-info.
* playbook wait step — same probe instead of /health.
* verify /health → verify /api/model-info reports loaded.
* verify /v1/audio/voices — switched from greping for `voice|alloy|echo`
literals to parsing JSON and asserting the actual response shape:
`{"status":"ok","voices":[...]}` (devnen's shape — note this is NOT
the OpenAI list-format vibevoice uses).
This commit is contained in:
@@ -70,22 +70,32 @@ steps:
|
|||||||
- name: docker compose up -d
|
- name: docker compose up -d
|
||||||
shell: cd {{ compose_dir }} && docker compose up -d
|
shell: cd {{ compose_dir }} && docker compose up -d
|
||||||
|
|
||||||
- name: Wait for /health to respond (allow ~15 min for model download + warmup)
|
- name: Wait for /api/model-info to report loaded:true (allow ~15 min for model download + warmup)
|
||||||
|
# The devnen wrapper has no /health endpoint. /api/model-info returns
|
||||||
|
# `{"loaded":true,...}` only after the model finishes loading, so it
|
||||||
|
# doubles as a liveness + ready probe.
|
||||||
shell: |
|
shell: |
|
||||||
for i in $(seq 1 180); do
|
for i in $(seq 1 180); do
|
||||||
curl -sf -o /dev/null --max-time 3 http://localhost:{{ host_port }}/health && exit 0
|
curl -sf --max-time 3 http://localhost:{{ host_port }}/api/model-info \
|
||||||
|
| grep -q '"loaded":true' && exit 0
|
||||||
sleep 5
|
sleep 5
|
||||||
done
|
done
|
||||||
exit 1
|
exit 1
|
||||||
changed_when: "false"
|
changed_when: "false"
|
||||||
|
|
||||||
verify:
|
verify:
|
||||||
- name: /health returns 200
|
- name: /api/model-info reports loaded:true
|
||||||
shell: curl -sf -o /dev/null http://localhost:{{ host_port }}/health
|
shell: curl -sf http://localhost:{{ host_port }}/api/model-info | grep -q '"loaded":true'
|
||||||
changed_when: "false"
|
changed_when: "false"
|
||||||
|
|
||||||
- name: /v1/audio/voices returns valid JSON
|
- name: /v1/audio/voices returns the wrapper's status:ok + voices list
|
||||||
shell: curl -sf http://localhost:{{ host_port }}/v1/audio/voices | grep -q 'voice\|alloy\|echo'
|
# Devnen's response shape is `{"status":"ok","voices":["Abigail.wav",...]}`
|
||||||
|
# — voices is an array of files in the predefined-voices dir. Assert
|
||||||
|
# both fields rather than greping for any one voice name (the seed
|
||||||
|
# set could change with upstream).
|
||||||
|
shell: |
|
||||||
|
curl -sf http://localhost:{{ host_port }}/v1/audio/voices \
|
||||||
|
| python3 -c "import json,sys; d=json.load(sys.stdin); assert d.get('status')=='ok' and isinstance(d.get('voices'), list)"
|
||||||
changed_when: "false"
|
changed_when: "false"
|
||||||
|
|
||||||
- name: Container is running
|
- name: Container is running
|
||||||
|
|||||||
@@ -49,8 +49,11 @@ services:
|
|||||||
# is to use the in-image config + env var overrides.
|
# is to use the in-image config + env var overrides.
|
||||||
# - ${CHATTERBOX_CONFIG}:/app/config.yaml:ro
|
# - ${CHATTERBOX_CONFIG}:/app/config.yaml:ro
|
||||||
healthcheck:
|
healthcheck:
|
||||||
# Devnen wrapper exposes /health; the OpenAPI/docs path is /docs.
|
# The devnen wrapper does NOT expose /health (no such route — use
|
||||||
test: ["CMD-SHELL", "curl -fsS -o /dev/null http://localhost:8004/health || exit 1"]
|
# /docs or /openapi.json to enumerate). /api/model-info returns
|
||||||
|
# `{"loaded":true,...}` only after the model finishes loading,
|
||||||
|
# so it doubles as a liveness + ready probe.
|
||||||
|
test: ["CMD-SHELL", "curl -fsS http://localhost:8004/api/model-info | grep -q '\"loaded\":true' || exit 1"]
|
||||||
interval: 30s
|
interval: 30s
|
||||||
timeout: 10s
|
timeout: 10s
|
||||||
retries: 3
|
retries: 3
|
||||||
|
|||||||
Reference in New Issue
Block a user