Skip to content

feat(templates): add TailAdmin Voice Core template - #2

Open
sinan-aydogan wants to merge 1 commit into
Panelica:mainfrom
sinan-aydogan:add-tailadmin-voice-core-template
Open

sinan-aydogan wants to merge 1 commit into
Panelica:mainfrom
sinan-aydogan:add-tailadmin-voice-core-template

Conversation

@sinan-aydogan

Copy link
Copy Markdown
Contributor

New template: TailAdmin Voice Core

Checklist

  • python3 scripts/validate.py prints OK
  • I deployed this template myself and reached the app's first-run screen
  • min_memory_mb reflects real usage (checked with docker stats)
  • Secrets use "secret": true with an empty default (auto-generated)
  • Data-service ports (db/cache/queue) bind "host_ip": "127.0.0.1"

Test notes

  • Multi-container AI stack:
    • Main app: sinanaydogan/voice-core-app (Inertia Vue 3 Dashboard + REST API + Laravel Queue Worker + SQLite).
    • Sidecar engine: sinanaydogan/voice-core-engine (FastAPI + PyTorch AI Engine for Piper TTS, XTTS v2, Bark, and Faster-Whisper).
  • Inter-container network: App connects to the Python engine via http://{{slug}}-engine:5001.
  • Web UI & API port: 8000. Engine port 5001 stays strictly on the internal stack network.
  • Persistent volumes: {{slug}}_data for models and audio outputs, {{slug}}_database for SQLite storage.
  • Auto-generated secrets: APP_KEY and VOICE_CORE_API_KEY are safely generated by Panelica.
  • Validation: Passed python scripts/validate.py with 0 errors.

Copilot AI lite review requested due to automatic review settings September 18, 2026 13:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Panelica Panelica left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the submission. The template itself is valid (schema, lint and CI all pass), but we deploy every template on a clean VM before merging, and both containers of this stack crash-loop with the images currently published on Docker Hub (voice-core-engine:latest and voice-core-app:latest, pushed 2026-09-21). Neither problem can be fixed from the template side, so we are holding this PR until the images are fixed.

1. Engine container (sinanaydogan/voice-core-engine) exits immediately

File "/app/engine/main.py", line 272, in <module>
    app.mount("/data", StaticFiles(directory="data"), name="data")
RuntimeError: Directory 'data' does not exist

WORKDIR is /app/engine, so the relative "data" resolves to /app/engine/data, which does not exist in the image (the data volume is mounted at /app/data). Using an absolute path (e.g. /app/data) or creating the directory before mounting fixes it.

2. App container (sinanaydogan/voice-core-app) exits during the entrypoint

Starting Laravel Queue Worker in background...
trap: SIGTERM: bad trap

docker-entrypoint.sh runs under #!/bin/sh (dash on Debian) with set -e. dash does not accept the SIG prefix in trap, so trap cleanup SIGTERM SIGINT fails and the script exits before php artisan serve starts. Use trap cleanup TERM INT (POSIX) or switch the shebang to #!/bin/bash.

Other points to address before we can merge

  • min_memory_mb: 2048 does not look realistic for XTTS v2 / Bark on CPU; please measure a real synthesis run with docker stats and set the value accordingly. The engine image is 4.3 GB compressed / 14 GB on disk, which is also worth stating in post_install_notes.
  • Please add a LICENSE file to the upstream repo and use English for description and post_install_notes (the catalog is the English source for 31 languages).
  • Consider publishing version tags and pinning default_tag.

Once new images are pushed, comment here and we will re-test on the VM and merge. Happy to help if anything above is unclear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants