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

[BUGFIX] Augmente la durée maximum du job de calcul de la certificabilité. #11430

Conversation

HEYGUL
Copy link
Contributor

@HEYGUL HEYGUL commented Feb 16, 2025

🥞 Problème

Suite au changement du calcul de certificabilité pour le rendre synchrone, le job, qui s'occupait auparavant uniquement de planifier les calculs, dépasse le temps par défaut de 15 minutes.

🥓 Proposition

On modifie le constructeur du JobScheduleController pour accepter comme option un expireIn permettant de modifier la durée d'expiration d'un job planifié.
On ajuste cette option à 4h pour le job de calcul de la certificabilité.

🧃 Remarques

😋 Pour tester

A voir avec @1024pix/team-prescription

@HEYGUL HEYGUL self-assigned this Feb 16, 2025
@HEYGUL HEYGUL requested a review from a team as a code owner February 16, 2025 11:41
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@HEYGUL HEYGUL added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally team-prescription team-captains This is your captain speaking and removed Development in progress labels Feb 16, 2025
@HEYGUL HEYGUL added 🚀 Ready to Merge and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Feb 17, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the fix-allow-schedule-job-to-define-its-own-expiration-delay branch from b808787 to ef8262f Compare February 17, 2025 14:51
@pix-service-auto-merge pix-service-auto-merge merged commit d2bce27 into dev Feb 17, 2025
8 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the fix-allow-schedule-job-to-define-its-own-expiration-delay branch February 17, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants