fix(tts): sync_audio_to_prod.py respeta fea_audio_skip (issue #190) #218

Open
rafa wants to merge 1 commits from fix/tts-audio-sync-donations-guard-190 into main
Owner

Qué corrige

sync_audio_to_prod.py solo comprobaba fea_audio_done antes de subir audio a
producción, ignorando el flag fea_audio_skip que el hotfix de #190 introdujo
en tts_produce.py (el script de generación). Cualquier post que ya tuviera
fea_audio_done=1 localmente de antes de ese hotfix (ej. #18036, la página
de cuentas/donaciones "numeros") volvía a publicarse en cuanto entrara en el
cluster de una carta cualquiera.

Pasó hoy: sync_audio_to_prod.py --carta 56290 (despliegue de la carta 741)
restauró en prod el mp3 de 3h27/190 MiB de #18036 — exactamente el incidente
que #190 daba por cerrado. Revertido manualmente (--rollback --ids 18036 +
limpieza del meta local obsoleto); este PR cierra el hueco de raíz.

Cambio

Una comprobación adicional al principio de sync_one(): si fea_audio_skip=1,
se excluye antes de mirar fea_audio_done. Sin cambios de comportamiento para
el resto de posts.

De paso, documenta el alcance correcto del TTS (solo ES, un mp3 por artículo,
nunca multimedia) en docs/guia-tts-traduccion-inma.md §4, para Inma.

Prueba

Dry-run contra el cluster real de la carta 741 (30 posts, incluye #18036):

#18036: excluido (fea_audio_skip)

Referencia: #190, #216

## Qué corrige `sync_audio_to_prod.py` solo comprobaba `fea_audio_done` antes de subir audio a producción, ignorando el flag `fea_audio_skip` que el hotfix de #190 introdujo en `tts_produce.py` (el script de generación). Cualquier post que ya tuviera `fea_audio_done=1` localmente de *antes* de ese hotfix (ej. `#18036`, la página de cuentas/donaciones "numeros") volvía a publicarse en cuanto entrara en el cluster de una carta cualquiera. Pasó hoy: `sync_audio_to_prod.py --carta 56290` (despliegue de la carta 741) restauró en prod el mp3 de 3h27/190 MiB de `#18036` — exactamente el incidente que #190 daba por cerrado. Revertido manualmente (`--rollback --ids 18036` + limpieza del meta local obsoleto); este PR cierra el hueco de raíz. ## Cambio Una comprobación adicional al principio de `sync_one()`: si `fea_audio_skip=1`, se excluye antes de mirar `fea_audio_done`. Sin cambios de comportamiento para el resto de posts. De paso, documenta el alcance correcto del TTS (solo ES, un mp3 por artículo, nunca multimedia) en `docs/guia-tts-traduccion-inma.md` §4, para Inma. ## Prueba Dry-run contra el cluster real de la carta 741 (30 posts, incluye `#18036`): ``` #18036: excluido (fea_audio_skip) ``` Referencia: #190, #216
rafa added 1 commit 2026-08-26 18:16:30 +00:00
sync_one() solo comprobaba fea_audio_done, ignorando fea_audio_skip. El
guard de #190 se anadio a tts_produce.py (generacion) pero nunca a este
script (subida a prod), asi que cualquier post con audio local obsoleto
de antes del hotfix (ej. #18036, pagina de cuentas/donaciones) volvia a
publicarse en cuanto entrara en el cluster de una carta. Paso hoy con la
carta 741: sync_audio_to_prod.py --carta 56290 restauro el mp3 de 3h27
en produccion. Revertido manualmente; este commit cierra el hueco.

Tambien documenta el alcance correcto del TTS (solo ES, un mp3 por
articulo, nunca multimedia) en la guia de Inma.

Co-authored-by: rafa <rcalvotorrejon@gmail.com>
Signed-off-by: rafa <rcalvotorrejon@gmail.com>
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/tts-audio-sync-donations-guard-190:fix/tts-audio-sync-donations-guard-190
git checkout fix/tts-audio-sync-donations-guard-190
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: rafa/feadulta#218