envoyproxy/envoy

[http] Further improvements to Envoy crash logging.

Open

#11 432 ouverte le 3 juin 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/envoy_loghelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

Ref: #7300

We have some logging of the active request on crashes under callstacks that have a ScopeTrackerScopeState but those scoped objects are only associated with events under some HCM, UpstreamRequest and timer callstacks. Also, dumped information is fairly limited.

Possible improvements to ScopeTrackerScopeState to increase dump coverage:

  • Have the FileEventImpl or network::ConnectionImpl hold a pointer to the ScopeTrackedObject associated with the current H1 request or active H2 connection. Add a ScopeTrackerScopeState to ConnectionImpl::onFileEvent/FileEventImpl that calls the dump method on the active request's ScopeTrackedObject
  • Implement a ScopeTrackedObject for H2 connections that dumps basic info about active streams and recent stream/connection level events.

Possible improvements to request dumps:

  • LOG endpoints of the downstream and upstream connection
  • If in the middle of H1 header parsing, dump info about contents of read buffer (up to a limit, probably only the first buffer slice)

Guide contributeur

Direction de recherche
Examinez le mécanisme ScopeTrackerScopeState et de vidage existant dans Envoy. Déterminez comment l'intégrer à FileEventImpl et ConnectionImpl pour les connexions H1/H2. Étendez la connexion H2 pour implémenter ScopeTrackedObject afin de vider les flux et événements actifs.
Stack technique
cpp
Domaine
backendobservability
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Plus d'une semaine
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Investigation requise
Prérequis
C++EnvoyHTTP/2
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30