erplibre/test/test_warmup_http.py
Mathieu Benoit 46eebf0859 [ADD] run : précharger le registre Odoo au lancement, sans nuire
Odoo ne charge le registre d'une base qu'à la PREMIÈRE requête : sur une
base migrée, la personne qui ouvre la page attend des dizaines de secondes.
La sonde prend ce temps à sa place et s'arrête à la première réponse : un
303, un 404 ou un 500 prouvent tous que le registre est chargé.

Elle ne peut pas nuire : elle rend TOUJOURS 0, meurt avec run.sh par un trap
et abandonne après deux minutes. Le port vient de la ligne de commande, puis
de config.conf, puis du journal, exact même quand le port demandé était pris ;
l'écoute sur une interface vide ou générale est sondée par le bouclage.

`--erplibre-disable-warmup-http` la coupe, et elle seule est RETIRÉE avant
odoo_bin.sh, qu'Odoo refuse ; `--no-http` et `--stop-after-init` aussi.

--- EN ---

Odoo only loads a database's registry on the FIRST request: on a migrated
database, whoever opens the page waits tens of seconds. The probe takes that
time instead and stops at the first answer: a 303, a 404 or a 500 all prove
the registry is loaded.

It cannot get in the way: it ALWAYS returns 0, dies with run.sh through a
trap and gives up after two minutes. The port comes from the command line,
then config.conf, then the log, exact even when the requested port was taken;
an empty or wildcard listen interface is probed on the loopback.

`--erplibre-disable-warmup-http` turns it off, and it alone is STRIPPED
before odoo_bin.sh, which Odoo rejects; `--no-http` and `--stop-after-init` too.

Assisted-by: Claude Opus 5
(cherry picked from commit d0943782029ac9a23830d8a8041fccbcac98b67d)
2026-08-29 02:11:04 -04:00

331 lines
12 KiB
Python

#!/usr/bin/env python3
# © 2021-2026 TechnoLibre (http://www.technolibre.ca)
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl)
"""Le réveil HTTP : il doit se taire, et ne jamais gêner.
Cette sonde tourne à côté du serveur, pas devant lui. Tout ce qu'elle
peut faire de mal, elle le ferait en parlant : un message d'erreur dans
le journal de `run.sh` ferait chercher une panne d'Odoo qui n'existe pas,
et un code de retour non nul ferait échouer un démarrage réussi.
D'où les deux propriétés que ce fichier épingle avant les autres : elle
rend TOUJOURS 0, et elle n'écrit qu'une ligne — l'adresse choisie, parce
que c'est la seule chose qui permette de comprendre après coup un réveil
qui n'a pas eu lieu.
Le troisième sujet est l'option elle-même. `--erplibre-disable-warmup-http`
doit être RETIRÉE avant `odoo_bin.sh` : Odoo meurt sur « no such option »,
et le démarrage entier tombe pour un confort.
"""
import io
import unittest
from pathlib import Path
from script.odoo import warmup_http as warmup
RUN_SH = Path(__file__).resolve().parent.parent / "run.sh"
class TestWhereItKnocks(unittest.TestCase):
"""Le port, dans l'ordre de ce qui fait autorité."""
def test_the_command_line_wins_over_everything(self):
for argv in (
["-p", "8090"],
["--http-port", "8090"],
["--http-port=8090"],
):
self.assertEqual(warmup.port_de_la_ligne(argv), 8090, argv)
def test_a_port_that_is_not_a_number_is_ignored(self):
self.assertIsNone(warmup.port_de_la_ligne(["-p", "huit-mille"]))
self.assertIsNone(warmup.port_de_la_ligne(["-p"]))
def test_without_anything_it_falls_back_on_the_default(self):
vrai = warmup.lib_analyse.read_config
warmup.lib_analyse.read_config = lambda *a, **k: {}
try:
self.assertEqual(
warmup.adresse(),
(warmup.HOTE_PAR_DEFAUT, warmup.PORT_PAR_DEFAUT),
)
finally:
warmup.lib_analyse.read_config = vrai
def test_an_empty_interface_means_ourselves(self):
"""`http_interface =` veut dire « toutes » ; 0.0.0.0 n'est pas une
adresse de destination."""
vrai = warmup.lib_analyse.read_config
for valeur in ("", "False", "0.0.0.0", "::"):
warmup.lib_analyse.read_config = lambda *a, v=valeur, **k: {
"http_interface": v
}
try:
self.assertEqual(warmup.adresse()[0], "127.0.0.1", valeur)
finally:
warmup.lib_analyse.read_config = vrai
def test_a_real_interface_is_used_as_is(self):
vrai = warmup.lib_analyse.read_config
warmup.lib_analyse.read_config = lambda *a, **k: {
"http_interface": "10.0.0.5",
"http_port": "8071",
}
try:
self.assertEqual(warmup.adresse(), ("10.0.0.5", 8071))
finally:
warmup.lib_analyse.read_config = vrai
class TestReadingTheExecutionLog(unittest.TestCase):
"""La seule source exacte quand le port demandé était déjà pris."""
def _journal(self, contenu):
import tempfile
handle = tempfile.NamedTemporaryFile(
mode="w", suffix=".log", delete=False, encoding="utf-8"
)
handle.write(contenu)
handle.close()
self.addCleanup(lambda: Path(handle.name).unlink(missing_ok=True))
return handle.name
def test_it_reads_the_address_odoo_announced(self):
chemin = self._journal(
"blah\nHTTP service (werkzeug) running on 127.0.0.1:8075\nblah\n"
)
self.assertEqual(
warmup.adresse_du_journal(chemin), ("127.0.0.1", 8075)
)
def test_the_last_line_wins(self):
"""Un redémarrage laisse les deux ; la dernière est la vraie."""
chemin = self._journal(
"HTTP service (werkzeug) running on 127.0.0.1:8069\n"
"HTTP service (werkzeug) running on 127.0.0.1:8070\n"
)
self.assertEqual(warmup.adresse_du_journal(chemin)[1], 8070)
def test_a_missing_or_silent_log_says_nothing(self):
self.assertEqual(warmup.adresse_du_journal(None), (None, None))
self.assertEqual(warmup.adresse_du_journal("/pas/la"), (None, None))
self.assertEqual(
warmup.adresse_du_journal(self._journal("rien\n")), (None, None)
)
def test_the_command_line_still_wins_over_the_log(self):
chemin = self._journal(
"HTTP service (werkzeug) running on 127.0.0.1:8075\n"
)
self.assertEqual(
warmup.adresse(["-p", "8090"], journal=chemin)[1], 8090
)
class TestWhichDatabase(unittest.TestCase):
def test_the_one_run_sh_just_chose(self):
self.assertEqual(warmup.base_a_reveiller(["-d", "ma_base"]), "ma_base")
self.assertEqual(
warmup.base_a_reveiller(["--database=ma_base"]), "ma_base"
)
def test_otherwise_the_one_odoo_would_use(self):
vrai = warmup.lib_analyse.read_config
warmup.lib_analyse.read_config = lambda *a, **k: {"db_name": "prod"}
try:
self.assertEqual(warmup.base_a_reveiller([]), "prod")
finally:
warmup.lib_analyse.read_config = vrai
def test_db_name_false_is_not_a_database(self):
vrai = warmup.lib_analyse.read_config
for valeur in ("False", "", "None"):
warmup.lib_analyse.read_config = lambda *a, v=valeur, **k: {
"db_name": v
}
try:
self.assertIsNone(warmup.base_a_reveiller([]), valeur)
finally:
warmup.lib_analyse.read_config = vrai
def test_the_name_is_escaped_in_the_url(self):
self.assertIn("db=ma%20base", warmup.url("127.0.0.1", 8069, "ma base"))
def test_without_a_database_the_url_carries_no_parameter(self):
self.assertEqual(
warmup.url("127.0.0.1", 8069), "http://127.0.0.1:8069/web/login"
)
class TestWhenItStops(unittest.TestCase):
"""N'importe quelle réponse suffit ; seul le refus fait recommencer."""
def _sonder(self, reponses, delai=10.0):
import urllib.error
import urllib.request
essais = []
temps = [0.0]
def faux_urlopen(cible, timeout=None):
essais.append(cible)
resultat = (
reponses.pop(0) if reponses else ConnectionRefusedError()
)
if isinstance(resultat, Exception):
raise resultat
return io.BytesIO(b"")
vrai = urllib.request.urlopen
urllib.request.urlopen = faux_urlopen
try:
venu = warmup.sonder(
"http://x",
delai_total=delai,
entre_essais=0,
horloge=lambda: temps[0],
dormir=lambda s: temps.__setitem__(0, temps[0] + 1),
)
finally:
urllib.request.urlopen = vrai
return venu, essais
def test_a_first_answer_ends_it(self):
venu, essais = self._sonder([None])
self.assertTrue(venu)
self.assertEqual(len(essais), 1)
def test_an_http_error_is_an_answer(self):
"""303, 404, 500 : tous prouvent que le registre est chargé."""
import urllib.error
erreur = urllib.error.HTTPError("u", 500, "boom", {}, None)
venu, essais = self._sonder([erreur])
self.assertTrue(venu)
self.assertEqual(len(essais), 1)
def test_a_refused_connection_makes_it_try_again(self):
venu, essais = self._sonder([ConnectionRefusedError(), None])
self.assertTrue(venu)
self.assertEqual(len(essais), 2)
def test_it_gives_up_at_the_deadline(self):
venu, essais = self._sonder([], delai=5.0)
self.assertFalse(venu)
self.assertEqual(len(essais), 5)
def test_the_default_deadline_is_two_minutes(self):
self.assertEqual(warmup.DELAI_TOTAL, 120.0)
class TestItNeverGetsInTheWay(unittest.TestCase):
"""La propriété qui compte le plus : elle ne peut pas nuire."""
def test_it_always_returns_zero(self):
vrai = warmup.sonder
warmup.sonder = lambda *a, **k: (_ for _ in ()).throw(
RuntimeError("x")
)
try:
self.assertEqual(warmup.main(["--quiet"]), 0)
finally:
warmup.sonder = vrai
def test_a_broken_config_does_not_stop_it(self):
vrai = warmup.lib_analyse.read_config
warmup.lib_analyse.read_config = lambda *a, **k: (_ for _ in ()).throw(
OSError("illisible")
)
try:
self.assertEqual(warmup.main(["--quiet"]), 0)
finally:
warmup.lib_analyse.read_config = vrai
def test_it_announces_the_address_it_chose(self):
"""La seule ligne qu'elle écrit, et celle qui rend le silence
compréhensible plus tard."""
import contextlib
vrai = warmup.sonder
warmup.sonder = lambda *a, **k: True
sortie = io.StringIO()
try:
with contextlib.redirect_stdout(sortie):
warmup.main(["-d", "ma_base", "--", "-p", "8090"])
finally:
warmup.sonder = vrai
texte = sortie.getvalue()
self.assertEqual(len(texte.strip().splitlines()), 1)
self.assertIn("ma_base", texte)
def test_an_option_it_does_not_know_cannot_kill_it(self):
"""Les arguments d'Odoo nous traversent : l'un d'eux ressemblera un
jour à une option inconnue, et `parse_args` sortirait par SystemExit.
"""
vrai = warmup.sonder
warmup.sonder = lambda *a, **k: True
try:
self.assertEqual(warmup.main(["--quiet", "--option-a-odoo"]), 0)
finally:
warmup.sonder = vrai
def test_quiet_says_nothing_at_all(self):
import contextlib
vrai = warmup.sonder
warmup.sonder = lambda *a, **k: True
sortie = io.StringIO()
try:
with contextlib.redirect_stdout(sortie):
warmup.main(["--quiet"])
finally:
warmup.sonder = vrai
self.assertEqual(sortie.getvalue(), "")
class TestHowRunShWiresIt(unittest.TestCase):
"""Une option qu'Odoo ne connaît pas le tue au démarrage."""
@classmethod
def setUpClass(cls):
cls.source = RUN_SH.read_text(encoding="utf-8")
def test_the_flag_is_stripped_and_never_forwarded(self):
bloc = self.source[
self.source.index("--erplibre-disable-warmup-http)") :
][:120]
self.assertNotIn("EL_ARGS+=", bloc)
def test_no_http_disables_it_but_still_reaches_odoo(self):
"""Rien à réveiller, mais Odoo doit garder son option."""
bloc = self.source[
self.source.index("--no-http|--stop-after-init)") :
][:200]
self.assertIn("EL_WARMUP=0", bloc)
self.assertIn('EL_ARGS+=("$1")', bloc)
def test_it_runs_in_parallel(self):
lancement = [
texte
for texte in self.source.splitlines()
if texte.strip().endswith("&") and "2>/dev/null" in texte
]
self.assertTrue(lancement, "le réveil ne part pas en arrière-plan")
def test_it_does_not_survive_the_server(self):
self.assertIn("trap", self.source)
self.assertIn("EL_WARMUP_PID", self.source)
def test_its_own_errors_are_swallowed(self):
self.assertIn("2>/dev/null &", self.source)
def test_odoo_arguments_are_separated_by_a_double_dash(self):
"""Sans `--`, un « -c » destiné à Odoo serait lu comme le nôtre."""
self.assertIn('-- "${EL_ARGS[@]}" 2>/dev/null &', self.source)
if __name__ == "__main__":
unittest.main()