Produits Entr'ouvert » w.c.s.

Perso j'ai toujours bien du mal avec logging et comment ça se configure mal (je trouve), on peut faire avec mais qu'il y ait en début d'exécution la configuration du logging pour de forcer taper quelque chose qui serait

            'class': 'logging.handlers.TimedRotatingFileHandler',
            'filename': '/var/lib/wcs/logs/cron.log',
            'when': 'w0',

pour rendre la configuration indépendante de ce qui pourrait arriver via une configuration de LOGGING ?

Ça me va alternativement de faire totalement sans logging, avec juste

+    def log(self, message):
+        self.logger.info('[tenant %s] %s', get_publisher().tenant.hostname, message)

transformé pour faire

with open(..., 'a+') as fd:
    fd.write('la date et le tenant et le message\n')

Frédéric Péters a écrit :

Perso j'ai toujours bien du mal avec logging et comment ça se configure mal (je trouve), on peut faire avec mais qu'il y ait en début d'exécution la configuration du logging pour de forcer taper quelque chose qui serait

Juste un disclaimer : on a un TimedRotatingHandler maintenant pour les logs /var/log/brique/debug, je l'ai utilisé récemment pour débugger l'authentification OIDC sur l'instance intranet du GL (#57517), et ça marche très bizarrement avec uwsgi, la rotation du fichier est faite par un peu n'importe quel worker, donc on se retrouve avec un worker loggant vers debug et un autre vers debug.2021-10-11. Je déconseille vraiment, le plus sûr/simple ça reste de balancer ça sur stdout pour systemd (sans timestamp systemd/syslog l'ajouteront).

PS: et si /var/lib/wcs et sur un NFS c'est encore pire, le mode append ne marche pas avec NFS (et si besoin le soutien de SO https://stackoverflow.com/questions/41559260/djangouwsgi-logging-with-timedrotatingfilehandler-overwrites-rotated-log-file).

Juste un disclaimer (...)

(on est sur un cron qui s'exécute sur un seul nœud et pas via uwsgi).

Ça m'irait qu'on utilise l'heure locale, ça sera plus simple à comparer avec ce qu'on voit dans l'ui.

                if 'job' in inspect.getfullargspec(job.function).args:

Je serais pour ajouter des **kwargs partout et ne pas ainsi avoir besoin d'utiliser inspect. (désolé ici je m'étais déjà fait cette remarque et j'avais oublié de l'écrire).

CRON_LOG_FILENAME = '/var/lib/wcs/cron.log'

Il faudrait ne pas mettre ça ainsi et se baser sur WcsPublisher.APP_DIR. (je me suis dit 2 minutes que l'erreur jenkins venait peut-être de là mais sans doute que non).

je me suis dit 2 minutes que l'erreur jenkins venait peut-être de là

très probablement, mais je n'avais pas eu le temps de creuser encore :)