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

[FO - Nouveaux formulaire] Soumission du formulaire sans le traitement des fichiers (asynchrone) #2245

Merged
merged 2 commits into from
Feb 16, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Feb 15, 2024

Ticket

#2240

Review app : https://histologe-staging-pr2245.osc-fr1.scalingo.io

Description

Les temps de traitements peuvent être assez longs lors de la soumission d'un draft

Changements apportés

  • Déplacement de la logique vers un message handler
  • Mise en place d'une queue dédiée et prioritaire pour le traitement des images et pdf
  • Mise à jour du container phpworker pour la prise en charge de la nouvelle queue
  • Mise à jour du worker scalingo

Pré-requis

Mettez à jour le container phpworker

docker compose build histologe_phpworker
make worker-stop

Tests

  • Soumettez un signalement avec quelques images et constatez que les temps de traitement sont très légèrement supérieurs aux requêtes précédentes avec un signalement sans images

  • Soumettez un signalement avec beaucoup d'images (> 20) et constatez que les temps de traitement sont très légèrement supérieurs à la requête précédente (voir requêtes 54) avec un signalement sans images
    image

  • Lancez des exports et vérifiez que votre file d'attente possède des messages high priority
    image

  • Lancez maintenant le worker make worker-start et vérifiez que la pile se vide. Une fois le message high traité, retournez sur le signalement

  • Retournez sur le signalement et vérifiez que vous ayez toutes vos images et documents
    image

  • Refaite un test avec le worker allumé cette fois ci

@sfinx13 sfinx13 force-pushed the bugfix/2240-handling-file-async branch 2 times, most recently from 117ddd2 to 23380ea Compare February 15, 2024 18:41
@@ -1,3 +1,3 @@
postdeploy: ./scripts/postdeploy.sh
worker: php bin/console messenger:consume async
worker: php bin/console messenger:consume async_priority_high async
Copy link
Collaborator Author

@sfinx13 sfinx13 Feb 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai priorisé le traitement des images et envoi des pdfs aux autres messages (sish, schs, et oilhi)
et le worker scalingo se mettra à jour
image

Comment on lines +24 to +25
App\Messenger\Message\PdfExportMessage: async_priority_high
App\Messenger\Message\SignalementDraftFileMessage: async_priority_high
Copy link
Collaborator Author

@sfinx13 sfinx13 Feb 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai priorisé le traitement des images et envoi des pdfs aux autres messages (sish, schs, et oilhi)

@sfinx13 sfinx13 force-pushed the bugfix/2240-handling-file-async branch from 20a7147 to d077862 Compare February 15, 2024 19:09
Copy link

sonarcloud bot commented Feb 15, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test et relecture OK

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK lecture et tests !

@emilschn emilschn merged commit ac8a73d into main Feb 16, 2024
4 checks passed
@sfinx13 sfinx13 deleted the bugfix/2240-handling-file-async branch February 21, 2024 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants