Pas de création des tables lors d'une primo-installation

[klorydryk]

Pas de création des tables lors d'une primo-installation Pas d'ajout des nouvelles tables lors d'une mise à jour

Message de Benjamin :

je viens de tester la maj de cv-bretagne en 1.1.2 mais toujours des problèmes :

sqlalchemy.exc.ProgrammingError: (psycopg2.errors.UndefinedTable) relation "user" does not exist

LINE 2: FROM "user"
             ^
[SQL: SELECT "user".id AS user_id, "user".email AS user_email, "user".given_name AS user_given_name, "user".family_name AS user_family_name, "user".nc_locator AS user_nc_locator, "user".nc_login AS user_nc_login, "user".nc_token AS user_nc_token, "user".nc_last_auto_enroll AS user_nc_last_auto_enroll, "user".last_connection_utc_datetime AS user_last_connection_utc_datetime
FROM "user"
WHERE "user".email = %(email_1)s
 LIMIT %(param_1)s]
[parameters: {'email_1': '[benjamin.houry@ac-toulouse.fr](mailto:benjamin.houry@ac-toulouse.fr)', 'param_1': 1}]
(Background on this error at: https://sqlalche.me/e/14/f405)
[2023-12-20 11:21:18 +0000] [9] [ERROR] Error handling request /welcome
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/sqlalchemy/engine/[base.py](http://base.py/)", line 1910, in _execute_context
    [self.dialect.do](http://self.dialect.do/)_execute(
  File "/usr/local/lib/python3.11/site-packages/sqlalchemy/engine/[default.py](http://default.py/)", line 736, in do_execute
    cursor.execute(statement, parameters)
psycopg2.errors.UndefinedTable: relation "user" does not exist
LINE 2: FROM "user"

[azmeuk]

Erreur reproduite ce matin en live avec partage d'écran.

On s'est aussi dit qu'il faudrait intégrer des instructions pour les primo-installations à la documentation.

[LoanR]

On a un suspect principal sur les problèmes de migration des primo installations : Il y a une erreur dans l'environnement du conteneur du service web. La variable FLASK_APP doit avoir la valeur b3desk, or dans le conteneur de visio-test, elle renvoie encore flaskr. Ce changement est nécessaire depuis le 13/12. Avec le git pull de la demo, la nouvelle version du dépôt a été récupérée, celle qui nécessite ce changement. C'est probablement pour ça que les déploiements faits en novembre ont bien fonctionné. On attend confirmation après redéploiement de visio-test avec la bonne variable.

[LoanR]

@BHoury Ce qu'on compte faire concernant cette issue :

• [x] récupérer la sortie d'erreur de la commande de migration flask. Actuellement, il semble qu'il n'y ait que la sortie standard qui écrive dans /var/log/flask-migrate.log. Avoir les erreurs nous aurait donné plus rapidement des pistes.
• [x] renseigner en dur la variable FLASK_APP. Il semble trop risqué de pouvoir donner une valeur à cette variable quand on sait qu'il n'y a que la valeur b3desk qui permet de faire tourner le service. Elle sera ajoutée dans les fichiers docker-compose
• [x] Ajouter quelques précisions dans la documentation de déploiement.

[LoanR]

La variable FLASK_APP est fixée dans le docker-compose de base, elle n'est plus nécessaire dans la configuration. 573537e