From 6bc2c3793f47a6c7f45d576432ccc1981e54779f Mon Sep 17 00:00:00 2001 From: Johan LEROY Date: Mon, 14 Sep 2026 14:28:49 +0200 Subject: [PATCH] fix(db): monte db/init fichier par fichier et coupe la telemetrie Monter le dossier ./db/init sur /docker-entrypoint-initdb.d remplacait le dossier de l'image au lieu de s'y ajouter. Les trois scripts d'init livres par timescaledb-ha disparaissaient sans aucun message : creation de l'extension dans template1, reglage par timescaledb-tune, et installation de timescaledb_toolkit. Verifie au demarrage : le dossier ne contenait que nos deux fichiers, et timescaledb_toolkit etait absent des bases. Monter chaque fichier separement retablit l'ordre attendu, verifie dans les journaux : 000, 001, 010, puis 100 et 110. TIMESCALEDB_TELEMETRY passe a off par defaut : l'image envoie sinon des statistiques d'usage a Timescale, ce qui ne va pas pour un deploiement on-premise. --- .env.example | 2 ++ db/README.md | 35 ++++++++++++++++++++--------------- docker-compose.yml | 6 +++++- 3 files changed, 27 insertions(+), 16 deletions(-) diff --git a/.env.example b/.env.example index 4d97678..a5fba5a 100644 --- a/.env.example +++ b/.env.example @@ -6,6 +6,8 @@ POSTGRES_PASSWORD=change_me POSTGRES_DB=enervision # 5432 est souvent deja pris par une autre base du poste. POSTGRES_PORT=5433 +# `basic` renvoie des statistiques d'usage a Timescale. +TIMESCALEDB_TELEMETRY=off APP_ENV=local APP_DEBUG=true diff --git a/db/README.md b/db/README.md index c21a87d..fd62d6f 100644 --- a/db/README.md +++ b/db/README.md @@ -12,23 +12,28 @@ Les migrations du schema applicatif expose par l'API vivent dans ## `init` ne rejoue jamais -Le dossier est monte sur `/docker-entrypoint-initdb.d`, dont PostgreSQL ne joue le +Ces scripts sont montes sur `/docker-entrypoint-initdb.d`, dont PostgreSQL ne joue le contenu qu'a la toute premiere initialisation, quand `PGDATA` est vide. Modifier ou -ajouter un script ensuite reste sans effet sur une base existante : +ajouter un script ensuite reste sans effet sur une base existante : il faut detruire +le volume, ce que fait `make db-reset`. -```bash -docker compose down -v && docker compose up -d db -``` +L'image apporte ses propres scripts dans ce dossier, et ils comptent : -L'image joue d'abord ses propres scripts (`000_`, `001_`, `010_`), dont un -`CREATE EXTENSION IF NOT EXISTS timescaledb_toolkit CASCADE` qui installe `timescaledb` -au passage dans `postgres`, `template1` et la base applicative. Nos fichiers sont -numerotes a partir de `100` pour passer apres, quelle que soit la locale de tri. +| Script | Origine | Role | +|---|---|---| +| `000_install_timescaledb.sh` | image | Cree l'extension dans `postgres`, `template1` et la base applicative, et fixe `timescaledb.telemetry_level`. | +| `001_timescaledb_tune.sh` | image | Lance `timescaledb-tune` sur la memoire et les CPU vus par le conteneur. | +| `010_install_timescaledb_toolkit.sh` | image | Ajoute `timescaledb_toolkit`. | +| `100-extensions.sql` | ce depot | Declare explicitement les extensions attendues. | +| `110-test-database.sql` | ce depot | Cree `enervision_test`, attendue par la suite de tests du backend. | -| Script | Role | -|---|---| -| `100-extensions.sql` | Declare explicitement les extensions attendues. | -| `110-test-database.sql` | Cree `enervision_test`, attendue par la suite de tests du backend. | +D'ou deux contraintes dans `docker-compose.yml`. Nos fichiers sont **montes un par un**, +et non par leur dossier : un montage de `./db/init` sur `/docker-entrypoint-initdb.d` +remplacerait le dossier de l'image au lieu de s'y ajouter, et ferait disparaitre les trois +scripts ci-dessus sans le moindre message. Ajouter un fichier ici impose donc d'ajouter +une ligne la-bas. Et leur numerotation commence a `100` pour passer apres `010`, y compris +en locale C ou un prefixe a deux chiffres se trierait avant. -Comme un bootstrap peut toujours avoir ete saute, c'est `/api/v1/health/ready` qui fait -foi : la sonde refuse de repondre 200 si l'extension n'est pas chargee. +Comme un bootstrap peut toujours avoir ete saute, deux gardes le rattrapent : +`/api/v1/health/ready` repond 503 si l'extension n'est pas chargee, et la premiere +revision Alembic refuse de s'appliquer. diff --git a/docker-compose.yml b/docker-compose.yml index e6854d8..d8569c9 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -1,5 +1,7 @@ # Piege : PGDATA de l'image timescaledb-ha vaut /home/postgres/pgdata/data, pas le chemin # habituel de l'image postgres. Monte ailleurs, le volume ne retient rien, sans erreur. +# Piege : db/init est monte fichier par fichier. Monter le dossier masquerait les scripts +# d'init de l'image, dont timescaledb-tune. Ajouter un fichier impose une ligne ici. name: enervision @@ -10,11 +12,13 @@ services: POSTGRES_USER: ${POSTGRES_USER:?} POSTGRES_PASSWORD: ${POSTGRES_PASSWORD:?} POSTGRES_DB: ${POSTGRES_DB:?} + TIMESCALEDB_TELEMETRY: ${TIMESCALEDB_TELEMETRY:-off} ports: - "${POSTGRES_PORT:-5433}:5432" volumes: - pgdata:/home/postgres/pgdata/data - - ./db/init:/docker-entrypoint-initdb.d:ro + - ./db/init/100-extensions.sql:/docker-entrypoint-initdb.d/100-extensions.sql:ro + - ./db/init/110-test-database.sql:/docker-entrypoint-initdb.d/110-test-database.sql:ro healthcheck: test: ["CMD-SHELL", "pg_isready -U $${POSTGRES_USER} -d $${POSTGRES_DB}"] interval: 10s