Add job workers

[n-b]

refs #2154

~~à tester sur la review app: ~~
* est-ce que les jobs sont bien dépilés en parallèle?
* est-ce que ça consomme plus de mémoire?

Par ailleurs, il y a de mémoire un problème pour l’accès au dashboard de super_admin/delayed_jobs sur les review apps, il va peut-être falloir régler ça en premier.

Edit: je n’ai pas réussi à faire marcher delayed_job avec plusieurs workers; une autre piste sera de faire tourner plusieurs instances scalingo, chacune par queue. En attendant, on peut aussi faire tourner plusieurs instances de jobs, tout simplement.

Ce n’est pas (plus) ce que tente de faire cette PR: j’ajoute simplement deux autres noms de queues, :sms_low et :mailers_low, avec une moindre priorité. En principe, ça devrait permettre aux autres jobs de ne pas attendre 20 minutes, le matin à 20 heures pendant que les notifications de rappel sont envoyées.

AVANT LA REVUE

• Préparer des captures de l’interface avant et après
• Nettoyer les commits pour faciliter la relecture
• Supprimer les éventuels logs de test et le code mort

REVUE

• Relecture du code
• Test sur la review app / en local

[yaf]

Ça me semble un bon premier pas.

Merci !

[n-b]

Remarque en passant, j’ai réalisé pourquoi ça ne fonctionnait pas quand j’essayais d’utiliser une commande avec plusieurs pools, de ce genre delayed_job --pool=mailers:sms:2 --pool=webhooks --pool*:2 start. Pour gérer les workers multiples, delayed_job doit être lancé avec start (et pas run), et crée un certains nombre de process fils, puis quitte le process parent. (il fait des daemons). Comme le process initial est quitté, du point de vue du container, la tâche est terminée et le container est alors éteint.

Il y a des gens qui se sont pencher sur le problème (qui ont fait un fork de delayed_job pour… faire des forks). Ça a l’air à peu près maintenu, ça vaut sans doute le coût d’essayer.

Autre piste, on peut aussi regarder s’il y a des options de configuration quand on appelle rake jobs:work au lieu de lancer delayed_job depuis le Procfile.

En attendant, on peut déjà merger comme ça, affiner les priorités ne peut pas faire de mal.

[johnnyshields]

Je suis le mainteneur du fork de Delayed Job. Il fonctionne bien en production pour des milliards des jobs, je recommande de l'utiliser. SVP également indiquer aux responsables de Delayed Job que vous aimeriez qu'ils fusionner le PR.