Skip to content

RG-T125 TTS docker build fix #424

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ucswift merged 1 commit into from
Jul 16, 2026
+26 −13
Merged

RG-T125 TTS docker build fix #424

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
39 changes: 26 additions & 13 deletions Web/Resgrid.Web.Tts/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,19 +22,24 @@ RUN dotnet publish "Resgrid.Web.Tts.csproj" -c Release -o /app/publish /p:UseApp

FROM build AS publish
ARG PIPER_VERSION=v1.2.0
ARG PIPER_VOICES_REVISION=e21c7de8d4eab79b902f0d61e662b3f21664b8d2
COPY --from=base /etc/passwd /tmp/base-passwd
COPY --from=base /etc/group /tmp/base-group
RUN apt-get update \
RUN set -eu; \
apt-get update \
&& apt-get install -y --no-install-recommends \
libespeak-ng1 \
ca-certificates \
curl \
xz-utils \
gzip \
passwd \
&& rm -rf /var/lib/apt/lists/* \
&& curl -fsSL --retry 3 --retry-delay 5 "https://johnvansickle.com/ffmpeg/releases/ffmpeg-release-amd64-static.tar.xz" -o /tmp/ffmpeg.tar.xz \
&& curl -fsSL "https://johnvansickle.com/ffmpeg/releases/ffmpeg-release-amd64-static.tar.xz.md5" -o /tmp/ffmpeg.tar.xz.md5 \
&& rm -rf /var/lib/apt/lists/*

RUN set -eu; \
echo "Downloading FFmpeg" \
&& curl -fsSL --retry 5 --retry-delay 5 --retry-all-errors "https://johnvansickle.com/ffmpeg/releases/ffmpeg-release-amd64-static.tar.xz" -o /tmp/ffmpeg.tar.xz \
&& curl -fsSL --retry 5 --retry-delay 5 --retry-all-errors "https://johnvansickle.com/ffmpeg/releases/ffmpeg-release-amd64-static.tar.xz.md5" -o /tmp/ffmpeg.tar.xz.md5 \
Comment on lines +25 to +42

@coderabbitai coderabbitai Bot Jul 16, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🩺 Stability & Availability | 🟠 Major | ⚡ Quick win

Ensure the FFmpeg installation path aligns with the Kubernetes configuration.

The Dockerfile installs the static FFmpeg binary to /usr/local/bin/ffmpeg (line 49). However, the downstream Kubernetes deployment configuration explicitly expects it at /usr/bin/ffmpeg.

If the base image does not already provide a system-level FFmpeg symlink or binary at /usr/bin/ffmpeg, the TTS service will fail when attempting to process audio in Kubernetes. Please verify this behavior and consider either updating the cp destination here to /usr/bin/ffmpeg or modifying the deployment YAML to match /usr/local/bin/ffmpeg.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@Web/Resgrid.Web.Tts/Dockerfile` around lines 25 - 42, Align the FFmpeg
installation destination in the Dockerfile’s download/install RUN block with the
Kubernetes deployment’s expected /usr/bin/ffmpeg path. Update the cp target from
/usr/local/bin/ffmpeg, or consistently adjust the deployment configuration to
use the installed path, ensuring the TTS service resolves the binary in
Kubernetes.

&& echo "$(awk '{print $1}' /tmp/ffmpeg.tar.xz.md5) /tmp/ffmpeg.tar.xz" | md5sum -c - \
&& rm -f /tmp/ffmpeg.tar.xz.md5 \
&& xz -d /tmp/ffmpeg.tar.xz \
Expand All @@ -45,18 +50,23 @@ RUN apt-get update \
&& ffprobe_path=$(find /tmp -name 'ffprobe' -type f -executable -not -path '*/lib/*' -print -quit) \
&& test -n "$ffprobe_path" && cp "$ffprobe_path" /usr/local/bin/ffprobe \
&& chmod +x /usr/local/bin/ffmpeg /usr/local/bin/ffprobe \
&& rm -rf /tmp/ffmpeg* \
&& mkdir -p /usr/local/share/piper-voices \
&& curl -fsSL --retry 3 --retry-delay 5 "https://github.com/rhasspy/piper/releases/download/${PIPER_VERSION}/piper_amd64.tar.gz" -o /tmp/piper.tar.gz \
&& rm -rf /tmp/ffmpeg*

RUN set -eu; \
echo "Downloading Piper ${PIPER_VERSION}" \
&& curl -fsSL --retry 5 --retry-delay 5 --retry-all-errors "https://github.com/rhasspy/piper/releases/download/${PIPER_VERSION}/piper_amd64.tar.gz" -o /tmp/piper.tar.gz \
&& tar -xzf /tmp/piper.tar.gz -C /tmp \
&& mv /tmp/piper/piper /usr/local/bin/piper \
&& chmod +x /usr/local/bin/piper \
&& mkdir -p /usr/local/lib \
&& find /tmp/piper -name '*.so*' -exec cp {} /usr/local/lib/ \; \
&& if [ -d /tmp/piper/espeak-ng-data ]; then cp -R /tmp/piper/espeak-ng-data /usr/share/; else ln -sf /usr/lib/x86_64-linux-gnu/espeak-ng-data /usr/share/espeak-ng-data; fi \
&& ldconfig \
&& rm -rf /tmp/piper /tmp/piper.tar.gz \
&& for f in \
&& rm -rf /tmp/piper /tmp/piper.tar.gz

RUN set -eu; \
mkdir -p /usr/local/share/piper-voices; \
for f in \
"en/en_US/norman/medium/en_US-norman-medium" \
"es/es_MX/claude/high/es_MX-claude-high" \
"sv/sv_SE/nst/medium/sv_SE-nst-medium" \
Expand All @@ -68,10 +78,13 @@ RUN apt-get update \
"ar/ar_JO/kareem/medium/ar_JO-kareem-medium" \
; do \
name=$(basename "$f"); \
curl -fsSL --retry 3 --retry-delay 5 "https://huggingface.co/rhasspy/piper-voices/resolve/main/${f}.onnx" -o "/usr/local/share/piper-voices/${name}.onnx"; \
curl -fsSL --retry 3 --retry-delay 5 "https://huggingface.co/rhasspy/piper-voices/resolve/main/${f}.onnx.json" -o "/usr/local/share/piper-voices/${name}.onnx.json"; \
done \
&& groupadd --gid 10001 appgroup \
echo "Downloading Piper voice ${name}"; \
curl -fsSL --retry 5 --retry-delay 5 --retry-all-errors "https://huggingface.co/rhasspy/piper-voices/resolve/${PIPER_VOICES_REVISION}/${f}.onnx" -o "/usr/local/share/piper-voices/${name}.onnx"; \
curl -fsSL --retry 5 --retry-delay 5 --retry-all-errors "https://huggingface.co/rhasspy/piper-voices/resolve/${PIPER_VOICES_REVISION}/${f}.onnx.json" -o "/usr/local/share/piper-voices/${name}.onnx.json"; \
done

RUN set -eu; \
groupadd --gid 10001 appgroup \
&& useradd --uid 10001 --gid appgroup --create-home --shell /usr/sbin/nologin appuser \
&& cp /tmp/base-passwd /tmp/app-passwd \
&& cp /tmp/base-group /tmp/app-group \
Expand Down
Loading