https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342019-12-03T11:30:57ZRedmine Entr’ouvertw.c.s. - Development #38159: caler le déclenchement des jobs "jump" sur le déclenchement des jobs "hourly"https://dev.entrouvert.org/issues/38159?journal_id=2043412019-12-03T11:30:57ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Assigné à</strong> mis à <i>Frédéric Péters</i></li></ul> w.c.s. - Development #38159: caler le déclenchement des jobs "jump" sur le déclenchement des jobs "hourly"https://dev.entrouvert.org/issues/38159?journal_id=2043502019-12-03T12:38:32ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/39518">0001-misc-align-jump-checks-with-the-hourly-cron-jobs-381.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/39518/0001-misc-align-jump-checks-with-the-hourly-cron-jobs-381.patch">0001-misc-align-jump-checks-with-the-hourly-cron-jobs-381.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>En bonus l'alignement de l'envoi des messages d'aggrégation (qu'on n'utilise jamais, mais qui déclenchés exclusivement à 06:00 auraient désormais pu passer à la trappe vu que les sauts ne sont plus à h00.</p> w.c.s. - Development #38159: caler le déclenchement des jobs "jump" sur le déclenchement des jobs "hourly"https://dev.entrouvert.org/issues/38159?journal_id=2043862019-12-03T14:51:57ZThomas Noël
<ul></ul><p>Mais ne faut-il pas alors aligner tous les cron_jobs avec un minutes= fixe ?</p>
<p>Genre ici :</p>
<pre>
wcs/formdef.py: get_publisher_class().register_cronjob(CronJob(clean_drafts,
wcs/formdef.py- name='clean_drafts',
wcs/formdef.py- days=[2], hours=[0], minutes=[0]))
wcs/formdef.py- # once a day, look for unused files
wcs/formdef.py: get_publisher_class().register_cronjob(CronJob(clean_unused_files,
wcs/formdef.py- name='clean_unused_files',
wcs/formdef.py- hours=[2], minutes=[0]))
</pre>
<p>On ajouterait donc ici un hourly=True, comme pour les messages d'aggregation.</p>
<p>Et sur les sessions et les nonces, passer en mode "ord(settings.SECRET_KEY[-1]) % 60") ici :</p>
<pre>
wcs/qommon/publisher.py: cls.register_cronjob(CronJob(cls.clean_sessions, minutes=range(0, 60, 5)))
wcs/qommon/publisher.py: cls.register_cronjob(CronJob(cls.clean_nonces, minutes=range(0, 60, 5)))
</pre>
<p>Sans oublier, même si ça ne compte pas en Publik (saml) :</p>
<pre>
wcs/qommon/ident/password.py: CronJob(handle_unused_accounts, minutes=[0], hours=[6]))
wcs/qommon/ident/password.py: CronJob(handle_expired_tokens, minutes=[10], hours=[6]))
</pre> w.c.s. - Development #38159: caler le déclenchement des jobs "jump" sur le déclenchement des jobs "hourly"https://dev.entrouvert.org/issues/38159?journal_id=2043972019-12-03T15:15:01ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/39557">0001-misc-align-jump-checks-with-the-hourly-cron-jobs-381.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/39557/0001-misc-align-jump-checks-with-the-hourly-cron-jobs-381.patch">0001-misc-align-jump-checks-with-the-hourly-cron-jobs-381.patch</a> ajouté</li></ul><blockquote>
<p>On ajouterait donc ici un hourly=True, comme pour les messages d'aggregation.</p>
</blockquote>
<p>Ceux-là, oui.</p>
<blockquote>
<p>Et sur les sessions et les nonces, passer en mode "ord(settings.SECRET_KEY[-1]) % 60") ici :</p>
</blockquote>
<p>Ici je dirais que c'est inutile, c'est appelé suffisamment souvent, pas besoin de se caler pile.</p>
<blockquote>
<p>Sans oublier, même si ça ne compte pas en Publik (saml) :</p>
</blockquote>
<p>Ici pareil un hourly, mais oui inutile.</p>
<p><sub>~~</sub></p>
<p>Patch revu et j'ai décidé de garder minutes=[0] même quand j'ai ajouté hourly, en prenant ainsi, malgré son nom, hourly comme un paramètre altérant les minutes, plutôt qu'un vrai truc qui dirait toutes les heures.</p> w.c.s. - Development #38159: caler le déclenchement des jobs "jump" sur le déclenchement des jobs "hourly"https://dev.entrouvert.org/issues/38159?journal_id=2044422019-12-03T22:50:09ZThomas Noël
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Impec.</p>
<p>Et j'avais oublié de dire : good catch.</p>
<p>(À un moment il faudra quand même je me replonge dans une gestion des cron plus intelligente, notamment avec copain systemd)</p> w.c.s. - Development #38159: caler le déclenchement des jobs "jump" sur le déclenchement des jobs "hourly"https://dev.entrouvert.org/issues/38159?journal_id=2044542019-12-04T07:48:16ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>
commit 6a549fb0533add08efdcfac3b22a50786e678476
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Dec 3 13:36:27 2019 +0100
misc: align jump checks with the "hourly" cron jobs (#38159)
</pre> w.c.s. - Development #38159: caler le déclenchement des jobs "jump" sur le déclenchement des jobs "hourly"https://dev.entrouvert.org/issues/38159?journal_id=2047032019-12-04T15:15:16ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul>