cleanup: usuń martwy stary mechanizm html2docx (flaga + docker pull), domyka #16#17
Merged
Merged
Conversation
Domyka #16. Poprzednio html2docx dzialal przez flage DJANGO_BPP_ENABLE_HTML2DOCX_IMAGE, ktora w `make pull`/`up` robila `docker pull iplweb/html2docx:latest` na host, by appserver mogl go odpalic przez `docker run` (docker.sock). Po iplweb/bpp#552 (fallback przez HTTP) i #16 (opcjonalny serwis compose z profilem) ten mechanizm jest martwy: nikt nie robi juz `docker run`, a obraz serwisu pulluje samo `docker compose pull/up` gdy profil `html2docx` jest aktywny. - mk/deployment.mk: usunieta zmienna DJANGO_BPP_ENABLE_HTML2DOCX_IMAGE i oba bloki `docker pull iplweb/html2docx` (w `pull` i `up`); poprawiony komentarz o prune (serwis, gdy wlaczony, jest UP przed prune -> obraz w uzyciu). - scripts/init-configs.sh: template instancyjnego .env - zamiast starej flagi zakomentowany DJANGO_BPP_HTML2DOCX_URL + instrukcja (COMPOSE_PROFILES). Usuniety ensure_env_var starej flagi (nie dopisujemy jej do nowych/istniejacych .env). - docs/eksploatacja/komendy.md: notka o prune zaktualizowana pod model serwisu. Weryfikacja: bash -n OK, isolated make parse OK, docker compose config valid (profil ON -> 1 serwis html2docx). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cel
Domyka #16. Ten PR wprowadził nowy, opcjonalny serwis
html2docx(profil),ale nie usunął starego mechanizmu, który po iplweb/bpp#552 jest martwy.
Co było martwe
Stary model: flaga
DJANGO_BPP_ENABLE_HTML2DOCX_IMAGEpowodowała, żemake pull/uprobiłodocker pull iplweb/html2docx:latestna host — po to,by appserver mógł odpalić kontener przez
docker run(przezdocker.sock).Po #552 (fallback przez HTTP, appserver bez Docker CLI) i #16 (serwis compose
z profilem) to jest bez sensu:
docker run,docker compose pull/up, gdy profilhtml2docxjest aktywny (
COMPOSE_PROFILES).Zostawienie flagi +
docker pull= mylące (dwa mechanizmy) i pobieranie obrazu,którego nic nie uruchamia.
Zmiany
mk/deployment.mk: usuniętaDJANGO_BPP_ENABLE_HTML2DOCX_IMAGEi obabloki
docker pull iplweb/html2docx(wpulliup); komentarz o prunepoprawiony (serwis, gdy włączony profilem, jest UP przed prune → obraz w użyciu).
scripts/init-configs.sh: w template instancyjnego.envzamiast starejflagi jest zakomentowany
DJANGO_BPP_HTML2DOCX_URL+ instrukcja (dwa opt-inkroki:
COMPOSE_PROFILES=html2docxw repo-.env+ odkomentowanie URL).Usunięty
ensure_env_varstarej flagi.docs/eksploatacja/komendy.md: notka o prune zaktualizowana pod modelserwisu (była: „obraz pobierany po prune").
Stara flaga w istniejących
.envjest nieszkodliwa (nikt jej już nie czyta),więc jej nie ruszamy ani nie usuwamy.
Weryfikacja
bash -n scripts/init-configs.shOK,make -f mk/deployment.mk -n pullparsuje się (recipe =docker compose pull),docker compose configvalid; profil ON → 1 serwishtml2docx, brak resztekstarej flagi w żywej logice.
🤖 Generated with Claude Code