Développement #21995
agent: capturer stdout et stderr des sous processus
0%
Description
Dans hobo/agent/worker/services, deux endroits où des sous processus sont créés mais où rien n'est fait avec leur stdout/stderr si leur exitcode est 0.
Un peu comme ici #8932, on pourrait printer vers stdout/stderr du processus parent.
Utile pour debugger au moins dans le cas d'un hobo_deploy, où l'exitcode n'est pour l'instant pas une grosse garantie du bon déroulement de l'opération.
Files
Related issues
History
Updated by Benjamin Dauvergne about 7 years ago
Il me semble que sans rien faie sans filer PIPE, tout est déjà branché sur stdout/stderr du processus parent.
Updated by Benjamin Dauvergne about 7 years ago
Benjamin Dauvergne a écrit :
Il me semble que sans rien faire (et donc sans filer PIPE), tout est déjà branché sur stdout/stderr du processus parent.
Updated by Emmanuel Cazenave almost 7 years ago
- File 0001-agent-redirect-subprocess-fd-to-parent-21995.patch 0001-agent-redirect-subprocess-fd-to-parent-21995.patch added
- Patch proposed changed from No to Yes
Je n'utilise pas la suggestion de Benjamin parce que sinon pas possible d'insérer stdout/stderr dans l'exception en cas d'erreur.
Updated by Emmanuel Cazenave almost 7 years ago
- Related to Bug #23840: wcs déployé sans DB popstgres added
Updated by Emmanuel Cazenave almost 7 years ago
- Status changed from Nouveau to En cours
- Assignee set to Emmanuel Cazenave
Up sur ce ticket, vraiment utile pour debugger des problèmes de déploiements.
Updated by Thomas Noël almost 7 years ago
J'ai du mal à voir où ces info vont partir sur un "vrai" hobo-agent lancé via supervisor (ce qu'on fait actuellement, bientôt plutôt uwsgi mais la question sera toujours là). J'essaye de voir sur combo.dev.
Updated by Thomas Noël almost 7 years ago
Le résultat est que toutes les sorties des commandes hobo_deploy/notify finissent dans /var/lib/hobo-agents/*.log sous la forme de WARNING. Ce qui me chiffone un peu, je suis comme ça.
Ça m'irait bien de rajouter un if getattr(settings, 'DEBUG', False): devant ces lignes, donc.
Updated by Emmanuel Cazenave almost 7 years ago
- Related to Développement #24658: worker : adaptation mode opératoire quand DEBUG = True added