Development #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.
Fichiers
Demandes liées
Historique
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Il me semble que sans rien faie sans filer PIPE, tout est déjà branché sur stdout/stderr du processus parent.
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
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.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Fichier 0001-agent-redirect-subprocess-fd-to-parent-21995.patch 0001-agent-redirect-subprocess-fd-to-parent-21995.patch ajouté
- Patch proposed changé de Non à Oui
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.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Lié à Bug #23840: wcs déployé sans DB popstgres ajouté
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Emmanuel Cazenave
Up sur ce ticket, vraiment utile pour debugger des problèmes de déploiements.
Mis à jour par Thomas Noël il y a presque 6 ans
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.
Mis à jour par Thomas Noël il y a presque 6 ans
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.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Lié à Development #24658: worker : adaptation mode opératoire quand DEBUG = True ajouté