envoyproxy/envoy

signal handling: refactor some names for clarity

Open

#11.052 aperta il 4 mag 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
help wantedtech debt

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

While working on https://github.com/lyft/envoy-mobile/issue/831 @mattklein123 suggested disabling signal handling. I originally thought he meant the signal handling envoy has for fatal signals using SignalActions, which didn't make much sense in the context we were working in. While I should have understood he meant signal handling for termination, it didn't help that we have several places to "disable signals", and we do so in bazel config, via compiler directives, and in code. We could clean this up and specify which settings work on fatal signals, which on termination signals, etc.

Guida contributor

Direzione di ricerca
Indaga il codice corrente di gestione dei segnali in Envoy, concentrandosi su SignalActions e la gestione dei segnali di terminazione. Identifica dove "disabilita segnali" è implementato nella configurazione Bazel, nelle direttive del compilatore e nel codice. Proponi costanti/funzioni rinominate per differenziare la gestione dei segnali fatali dai segnali di terminazione. Rivedi l'issue #831 per il contesto.
Tech stack
cpp
Dominio
infrastructure
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65