Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configurer plusieurs jobs en parallèle pour les workers #2231

Closed
n-b opened this issue Mar 8, 2022 · 0 comments · Fixed by #3338
Closed

Configurer plusieurs jobs en parallèle pour les workers #2231

n-b opened this issue Mar 8, 2022 · 0 comments · Fixed by #3338
Labels
tech ticket sans impact direct sur les fonctionnalités de l'app

Comments

@n-b
Copy link
Contributor

n-b commented Mar 8, 2022

Suite de #2154 et #2155.

On n’a aujourd’hui plus de problème de bouchon de workers, on a configuré de queues de différentes priorités. Par contre, on pourrait quand même faire en sorte d’exécuter plus d’un job à la fois. Ça permettrait de dépiler plus vite :)

En principe, delayed_job le permet, en changeant l’invocation comme ceci:

- delayed_job run
+ delayed_job --pool=mailers:sms:2 --pool=webhooks --pool*:2 start

Le problème est que delayed_job start crée des process daemons et quitte le process appelant. Dans un container, c’est un problème, puisque celui-ci quitte alors immédiatement. Il y a une PR ouverte depuis longtemps qui permettrait de régler ça: je pense qu’on peut utiliser ce fork de delayed_job sans trop de risques.

@yaf yaf added the tech ticket sans impact direct sur les fonctionnalités de l'app label Apr 15, 2022
@yaf yaf added this to the Tech milestone Jul 2, 2022
@mekaidmekaid mekaidmekaid moved this to Icebox in RDV Service Public Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech ticket sans impact direct sur les fonctionnalités de l'app
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants