Skip to content

Corrige la méthode warmup_cache

Sylvain Boulade requested to merge fix-warmup_cache into main

Contexte

Depuis cette PR j'ai une erreur en local lorsque je lance l'API insitu:

ERROR:    Traceback (most recent call last):
  File "/Users/sylvain/Code/beta.gouv/insitu/.venv/lib/python3.10/site-packages/starlette/routing.py", line 677, in lifespan
    async with self.lifespan_context(app) as maybe_state:
  File "/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/contextlib.py", line 199, in __aenter__
    return await anext(self.gen)
  File "/Users/sylvain/Code/beta.gouv/insitu/insitu/api/lifespan.py", line 47, in lifespan
    registre_indicateurs.warmup_cache()
  File "/Users/sylvain/Code/beta.gouv/insitu/insitu/indicateurs/registre.py", line 472, in warmup_cache
    indicateur.definition.sql_tables
AttributeError: 'IndicateurDefinitionSQL' object has no attribute 'definition'

ERROR:    Application startup failed. Exiting.

Contenu

Si je reviens à indicateur.sql_tables comme avant cela compile à nouveau correctement.

@ronan @l-vincent-l est-ce que l'un de vous pourrait regarder si vous avez la même erreur en local ? Dans le cas contraire est-ce que vous auriez une idée d'où peut venir cette erreur ?

Je voulais également double checker en prod mais j'ai l'impression que ces changements n'ont pas encore été déployés

Merge request reports