entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

toulouse_maelis: restart paid notification job every 5 minutes on failure (#87030) #469

Merged
bdauvergne merged 1 commits from wip/87030-toulouse-maelis-notifier-les-fac into main 2024-02-20 11:17:49 +01:00
Conversation 2 Commits 1 Files Changed 2 +58 -13
bdauvergne commented 2024-02-15 19:02:00 +01:00
Owner
Copy Link
No description provided.
bdauvergne added 1 commit 2024-02-15 19:02:00 +01:00
gitea/passerelle/pipeline/head There was a failure building this commit Details
d0675015f7 toulouse_maelis: restart paid notification job every 5 minutes on failure (#87030)
bdauvergne force-pushed wip/87030-toulouse-maelis-notifier-les-fac from d0675015f7 to 087ce7ccad 2024-02-15 19:10:27 +01:00 Compare
bdauvergne force-pushed wip/87030-toulouse-maelis-notifier-les-fac from 087ce7ccad to 7f440f3ef8 2024-02-15 19:29:26 +01:00 Compare
bdauvergne force-pushed wip/87030-toulouse-maelis-notifier-les-fac from 7f440f3ef8 to 3c293b9372 2024-02-15 19:36:13 +01:00 Compare
bdauvergne changed title from WIP: toulouse_maelis: restart paid notification job every 5 minutes on failure (#87030) to toulouse_maelis: restart paid notification job every 5 minutes on failure (#87030) 2024-02-15 19:47:50 +01:00
bdauvergne requested review from nroche 2024-02-15 20:12:55 +01:00
nroche approved these changes 2024-02-16 10:09:32 +01:00
nroche left a comment
Owner
Copy Link

Ack.
Sur une erreur applicative (SOAPFault) à priori ça ne sert à rien de retenter,
mais ça ne pose pas de problème non plus.

Ack. Sur une erreur applicative (SOAPFault) à priori ça ne sert à rien de retenter, mais ça ne pose pas de problème non plus.
bdauvergne force-pushed wip/87030-toulouse-maelis-notifier-les-fac from 3c293b9372 to cd5e247de9 2024-02-16 17:19:12 +01:00 Compare
bdauvergne commented 2024-02-16 17:21:03 +01:00
Author
Owner
Copy Link

Ack.
Sur une erreur applicative (SOAPFault) à priori ça ne sert à rien de retenter,
mais ça ne pose pas de problème non plus.

Ouaip, j'ai ajusté et le test qui va avec, actuellement on retente quand même pour toujours dans le cron hourly, le fait de renvoyer False ou True ne fait rien concernant le critère de rejeu dans le cron (qui se base uniquement sur l'absence de date de notification). À terme je pense qu'on peut se permettre de supprimer complètement le cron pour n'utiliser que le rejeu via les jobs.

> Ack. > Sur une erreur applicative (SOAPFault) à priori ça ne sert à rien de retenter, > mais ça ne pose pas de problème non plus. Ouaip, j'ai ajusté et le test qui va avec, actuellement on retente quand même pour toujours dans le cron hourly, le fait de renvoyer False ou True ne fait rien concernant le critère de rejeu dans le cron (qui se base uniquement sur l'absence de date de notification). À terme je pense qu'on peut se permettre de supprimer complètement le cron pour n'utiliser que le rejeu via les jobs.
bdauvergne requested review from nroche 2024-02-16 17:21:07 +01:00
bdauvergne force-pushed wip/87030-toulouse-maelis-notifier-les-fac from cd5e247de9 to 60a579c88d 2024-02-16 17:39:24 +01:00 Compare
nroche requested changes 2024-02-16 17:56:01 +01:00
tests/test_toulouse_maelis.py Outdated
@ -11786,1 +11822,3 @@
assert con.invoice_set.get(regie_id=102, invoice_id=30).status() == 'paid'
assert 'fails to notify' in caplog.text
assert 'stopping' in caplog.text
assert con.invoice_set.get(regie_id=102, invoice_id=30).status() == 'notified'
nroche commented 2024-02-16 17:51:19 +01:00
Owner
Copy Link

Ce champ c'était pour garder en tête que l'appel au WS de notification maélis avait échoué.

        if self.status() in ['paid', 'cancelled', 'removed']:
            # wcs demand can be triggered
            return 'triggering'
Ce champ c'était pour garder en tête que l'appel au WS de notification maélis avait échoué. ``` if self.status() in ['paid', 'cancelled', 'removed']: # wcs demand can be triggered return 'triggering' ```
nroche commented 2024-02-16 18:06:48 +01:00
Owner
Copy Link

Non, pardon on utilise ce champ pour notifier la demande wcs, mais sans ce préoccuper de savoir si maelis a reçu ou non la notification :

            if self.invoice.status() in ['paid', 'notified']:
                # related invoice is paid
                return 'paid'
Non, pardon on utilise ce champ pour notifier la demande wcs, mais sans ce préoccuper de savoir si maelis a reçu ou non la notification : ``` if self.invoice.status() in ['paid', 'notified']: # related invoice is paid return 'paid' ```
nroche marked this conversation as resolved
nroche approved these changes 2024-02-16 18:07:15 +01:00
nroche left a comment
Owner
Copy Link

Ack.

Ack.
bdauvergne force-pushed wip/87030-toulouse-maelis-notifier-les-fac from 60a579c88d to e3278e6416 2024-02-20 07:31:39 +01:00 Compare
bdauvergne force-pushed wip/87030-toulouse-maelis-notifier-les-fac from e3278e6416 to 91b92aeb44 2024-02-20 07:58:09 +01:00 Compare
bdauvergne commented 2024-02-20 08:05:33 +01:00
Author
Owner
Copy Link

Ack.

J'ai continué dans #87168 parce que pour moi notifier plus souvent est effectivement bien mais il y a pour moi une incohérence à notifier w.c.s. d'un succès avant d'avoir pu le faire vers Maelis et pareillement de ne pas notifier w.c.s. d'un souci, ça empêche les agents de gérer le souci, puisque par définition Maelis n'a pas l'information sur le problème, de sont point vue il y a juste un panier de réservation jamais payé.

> Ack. J'ai continué dans #87168 parce que pour moi notifier plus souvent est effectivement bien mais il y a pour moi une incohérence à notifier w.c.s. d'un succès avant d'avoir pu le faire vers Maelis et pareillement de ne pas notifier w.c.s. d'un souci, ça empêche les agents de gérer le souci, puisque par définition Maelis n'a pas l'information sur le problème, de sont point vue il y a juste un panier de réservation jamais payé.
bdauvergne merged commit 91b92aeb44 into main 2024-02-20 11:17:49 +01:00
bdauvergne deleted branch wip/87030-toulouse-maelis-notifier-les-fac 2024-02-20 11:17:49 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#469
No description provided.
Delete Branch "wip/87030-toulouse-maelis-notifier-les-fac"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?