Bug #2521
Les exceptions levés dans Publisher mais avant la définition de self.statsd provoque une exception dans finish_failed_request
Statut:
Fermé
Priorité:
Bas
Assigné à:
-
Version cible:
-
Début:
19 février 2013
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Planning:
Description
2013-02-19 11:01:17 [13682] [ERROR] Error processing request. Traceback (most recent call last): File "/usr/lib/pymodules/python2.7/gunicorn/workers/sync.py", line 71, in handle self.handle_request(req, client, addr) File "/usr/lib/pymodules/python2.7/gunicorn/workers/sync.py", line 101, in handle_request respiter = self.wsgi(environ, resp.start_response) File "/usr/lib/pymodules/python2.7/quixote/wsgi.py", line 36, in __call__ response = self.publisher.process_request(request) File "/usr/lib/pymodules/python2.7/quixote/publish.py", line 281, in process_request output = self.finish_failed_request() File "/home/bdauvergne/wd/EO/wcs/wcs.git/wcs/qommon/publisher.py", line 257, in finish_failed_request self.statsd.increment('failed-request') AttributeError: WcsPublisher instance has no attribute 'statsd'Je ne sais pas trop quel est la meilleurse solution:
- rendre conditionnel l'appel à
increment()
en vérifiant queself.statsd
est bien défini - ou bien défini self.statsd plus tôt dans le cycle de vie du publisher.
Fichiers
Historique
Mis à jour par Frédéric Péters il y a environ 11 ans
- Fichier 0001-publisher-do-not-fail-on-early-statsd-call-2521.patch 0001-publisher-do-not-fail-on-early-statsd-call-2521.patch ajouté
- Statut changé de Nouveau à Solution déployée
Mis à jour par Thomas Noël il y a environ 11 ans
Yep, le patch de Fred est une bonne solution. Push !
Mis à jour par Frédéric Péters il y a environ 11 ans
- Statut changé de Solution déployée à Résolu (à déployer)
Ça roule, hop.
Mis à jour par Frédéric Péters il y a plus de 10 ans
- Statut changé de Résolu (à déployer) à Fermé