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

toulouse-foederis: increase requests timeout (#89287) #512

Merged
bdauvergne merged 1 commits from wip/89287-foederis-timeout into main 2024-04-10 16:37:28 +02:00
Conversation 0 Commits 1 Files Changed 1 +1 -1
csechet commented 2024-04-09 11:28:49 +02:00
Owner
Copy Link
No description provided.
csechet added 1 commit 2024-04-09 11:28:50 +02:00
gitea/passerelle/pipeline/head This commit looks good Details
3538d06f6f toulouse-foederis: increase requests timeout (#89287)
tnoel approved these changes 2024-04-09 14:32:07 +02:00
passerelle/contrib/toulouse_foederis/models.py
@ -350,3 +350,3 @@
kwargs.setdefault('headers', {})['API-Key'] = self.api_key
url = self.url + path
response = self.requests.request(method, url, **kwargs)
response = self.requests.request(method, url, timeout=50, **kwargs)
tnoel commented 2024-04-09 14:32:04 +02:00
Owner
Copy Link

Ca va fonctionner, mais il y a des moments où plusieurs requêtes s’enchaînent, par exemple dans update_announces. Si chaque requête mange 30 secondes, ça va très vite dépasser le maximum qu'on autorise dans un worker (harakiri = 120).

Après :

  1. comme on est dans des mises à jour de référentiels, ça peut sans doute crasher...
  2. de toute façon c'est quasi impossible d'arriver à une gestion intelligente des timeout sur ces moments, le problème n'est clairement pas dans passerelle, la solution non plus :)
Ca va fonctionner, mais il y a des moments où plusieurs requêtes s’enchaînent, par exemple dans update_announces. Si chaque requête mange 30 secondes, ça va très vite dépasser le maximum qu'on autorise dans un worker (harakiri = 120). Après : 1. comme on est dans des mises à jour de référentiels, ça peut sans doute crasher... 2. de toute façon c'est quasi impossible d'arriver à une gestion intelligente des timeout sur ces moments, le problème n'est clairement pas dans passerelle, la solution non plus :)
bdauvergne merged commit 3538d06f6f into main 2024-04-10 16:37:28 +02:00
bdauvergne deleted branch wip/89287-foederis-timeout 2024-04-10 16:37:29 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
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#512
No description provided.
Delete Branch "wip/89287-foederis-timeout"

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?