Project

General

Profile

Development #86346

Le cache de RemoteTemplate lève trop d'erreurs inutiles

Added by Benjamin Dauvergne 4 months ago. Updated 9 days ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
30 January 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

On a régulièrement des traces sentry de ce type1:

HTTPSConnectionPool(host='demarches.ville-sens.fr', port=443): Read timed out. (read timeout=10)

ReadTimeout: HTTPSConnectionPool(host='demarches.ville-sens.fr', port=443): Read timed out. (read timeout=10)
  File "threading.py", line 892, in run
    self._target(*self._args, **self._kwargs)
  File "hobo/context_processors.py", line 116, in update_content
    r = requests.get(
  File "requests/api.py", line 76, in get
    return request('get', url, params=params, **kwargs)
  File "requests/api.py", line 61, in request
    return session.request(method=method, url=url, **kwargs)
  File "requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "requests/sessions.py", line 655, in send
    r = adapter.send(request, **kwargs)
  File "requests/adapters.py", line 529, in send
    raise ReadTimeout(e, request=request)

qui ne nous apporte pas grand choses à part du bruit.

Je propose d'augmenter de ne lever une erreur que si ça fait plus d'1 heure qu'une mise à jour n'a pas eu lieu.

1 https://sentry.entrouvert.org/entrouvert/publik/issues/113170/

Associated revisions

Revision 202f963a (diff)
Added by Benjamin Dauvergne 11 days ago

tests: use responses in context processor tests (#86346)

Revision d78e9954 (diff)
Added by Benjamin Dauvergne 11 days ago

context_processor: handle network errors in update_all_pages_cache (#86346)

Revision 3ca30d74 (diff)
Added by Benjamin Dauvergne 11 days ago

context_processor: trace if template refresh fails for 1 hour and 3 times (#86346)

Aslo capture network errors in the same error handling code.

History

#1

Updated by Benjamin Dauvergne 4 months ago

  • Description updated (diff)
#2

Updated by Robot Gitea 4 months ago

  • Status changed from Nouveau to En cours

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#3

Updated by Robot Gitea 4 months ago

  • Status changed from En cours to Solution proposée
#4

Updated by Robot Gitea 3 months ago

Benjamin Dauvergne (bdauvergne) a demandé une relecture de Thomas NOËL (tnoel) sur une pull request sur Gitea concernant cette demande :

#5

Updated by Robot Gitea 29 days ago

  • Status changed from Solution proposée to Solution validée

Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :

#6

Updated by Robot Gitea 11 days ago

  • Status changed from Solution validée to Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#7

Updated by Transition automatique 9 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF