Produits Entr'ouvert » Passerelle

Effectivement dans Django on a :

class UserSettingsHolder:
    """Holder for user configured settings.""" 
    # SETTINGS_MODULE doesn't make much sense in the manually configured
    # (standalone) case.
    SETTINGS_MODULE = None

    def __init__(self, default_settings):

et comme les settings des tenants c'est des UserSettingsHolder ça peut pas marcher (et pourtant ça a marché il me semble, et ce code n'a pas bougé entre Django 1.8 et Django 2.x).

discussion sur jabber: aller chercher le nom de l'appli via os.envirron('DJANGO_SETTINGS_MODULE').split('.')[0] ou settings.PROJECT_NAME (et faire en sorte que "authentic*" devienne "authentic")

J'ai redémarré passerelle après avoir ajouté un assert False dans django.conf.Settings.configure() j'attends que ça pète, le souci vient du fait que django.config.settings._wrapped se retrouve à contenir un object UserSettingsHolder mais je n'arrive pas à trouver d'où il vient, le seul code qui fasse ça c'est cette fonction django.conf.Settings.configure(), mais un grep dans /usr/lib/python2.7 ne me révèle aucun appelant à part dans les tests de kombu. Je remarque que le problème à commencer à 17:10 et dans la suite ça vient toujours du même pid, 693211.

Et.... il suffit que tu m'interpelles pour que je comprenne, ça vient certainement de ce code :

passerelle/base/models.py:from django.test import override_settings
passerelle/base/models.py:        with override_settings(ADMINS=admins):

Dec 21 17:10:11 passerelle uwsgi[693211]: Traceback (most recent call last):
Dec 21 17:10:11 passerelle uwsgi[693211]:   File "/usr/lib/python2.7/dist-packages/django/core/handlers/wsgi.py", line 154, in __call__
Dec 21 17:10:11 passerelle uwsgi[693211]:     set_script_prefix(get_script_name(environ))
Dec 21 17:10:11 passerelle uwsgi[693211]:   File "/usr/lib/python2.7/dist-packages/django/core/handlers/wsgi.py", line 188, in get_script_name
Dec 21 17:10:11 passerelle uwsgi[693211]:     if settings.FORCE_SCRIPT_NAME is not None:
Dec 21 17:10:11 passerelle uwsgi[693211]:   File "/usr/lib/python2.7/dist-packages/hobo/multitenant/apps.py", line 134, in <lambda>
Dec 21 17:10:11 passerelle uwsgi[693211]:     conf.LazySettings.__getattr__ = lambda self, name: getattr(self._wrapped, name)
Dec 21 17:10:11 passerelle uwsgi[693211]:   File "/usr/lib/python2.7/dist-packages/django/conf/__init__.py", line 172, in __getattr__
Dec 21 17:10:11 passerelle uwsgi[693211]:     return getattr(self.default_settings, name)
Dec 21 17:10:11 passerelle uwsgi[693211]:   File "/usr/lib/python2.7/dist-packages/hobo/multitenant/settings.py", line 76, in __getattr__
Dec 21 17:10:11 passerelle uwsgi[693211]:     return getattr(self.get_wrapped(), name)
Dec 21 17:10:11 passerelle uwsgi[693211]:   File "/usr/lib/python2.7/dist-packages/hobo/multitenant/settings.py", line 71, in get_wrapped
Dec 21 17:10:11 passerelle uwsgi[693211]:     return self.get_tenant_settings(self.default_settings, tenant)
Dec 21 17:10:11 passerelle uwsgi[693211]:   File "/usr/lib/python2.7/dist-packages/hobo/multitenant/settings.py", line 59, in get_tenant_settings
Dec 21 17:10:11 passerelle uwsgi[693211]:     wrapped, tenant, tenant_settings, last_time)
Dec 21 17:10:11 passerelle uwsgi[693211]:   File "/usr/lib/python2.7/dist-packages/hobo/multitenant/settings.py", line 44, in load_tenant_settings
Dec 21 17:10:11 passerelle uwsgi[693211]:     loader_instance.update_settings(tenant_settings, tenant)
Dec 21 17:10:11 passerelle uwsgi[693211]:   File "/usr/lib/python2.7/dist-packages/hobo/multitenant/settings_loaders.py", line 212, in update_settings
Dec 21 17:10:11 passerelle uwsgi[693211]:     module_name = settings.SETTINGS_MODULE.split('.')[0]
Dec 21 17:10:11 passerelle uwsgi[693211]: AttributeError: 'NoneType' object has no attribute 'split'
Dec 21 17:10:11 passerelle uwsgi[693211]: {address space usage: 1444126720 bytes/1377MB} {rss usage: 133840896 bytes/127MB} [pid: 693217|app: 0|req: 884/5244] 0.0.0.0 () {40 vars in 1115 bytes} [Fri Dec 21 17:10:11 2018] GET /atos-genesys/genesys-preprod/dossiers?orig=departement06.test.entrouvert.org&NameID=562eaec5c2914f52a2d592673e22c4c6&link_id=69&algo=sha256&timestamp=2018-12-21T16%3A10%3A11Z&nonce=1d77ad4316cac175c323a13e31ffe622L&signature=CNar5TZiIFvQsmO2cm/7fH7w0ZqopUQa1Q/YuMjkDbA%3D => generated 0 bytes in 11 msecs (HTTP/1.0 500) 0 headers in 0 bytes (0 switches on core 0)

Donc on a une requête qui arrive, pendant ce temps le thread de genesys fait des mises à jour du cache, qui font des appels HTTL via BaseResource.request et bim ça log une erreur, et bim ça fait override_settings() en plein milieu du traitement de la requête et ThemeSettings se retrouve avec un UserSettingsHolder sous les pieds et ça plante, une fois le override_settings() terminé tout est défait et ça remarche, d'où l'intermittence.

Je n'ai pas d'idée de fix pour l'instant, mais au moins on sait d'où ça vient.

Je vais juste virer ces histoires de threads dans atos-genesys.

Mais accéder à SETTINGS_MODULE reste une mauvaise idée vu que ça ne marche pas toujours (ça pourra péter dans des tests qui utiliseront override_settings()).