elastic/kibana

[Telemetry] Better error handling at the `telemetry_plugin_collector`

Open

#122.634 aperta il 11 gen 2022

Vedi su GitHub
 (4 commenti) (1 reazione) (0 assegnatari)TypeScript (8021 fork)batch import
Feature:TelemetryTeam:Coregood first issueimpact:lowloe:small

Metriche repository

Star
 (19.065 star)
Metriche merge PR
 (Merge medio 1g 16h) (999 PR mergiate in 30 g)

Descrizione

In the telemetry_plugin_collector we are swallowing any errors thrown by getTelemetrySavedObject: https://github.com/elastic/kibana/blob/c2641bd8cc875dd50c63a40fbaf3054780cd570b/src/plugins/telemetry/server/collectors/telemetry_plugin/telemetry_plugin_collector.ts#L45-L52

I don't think that we should do this as we might report different data depending on whether there's an ES connection issue or not (because when there's no savedObject information we rely on the kibana.yml config values).

IMO, we should either let the collector fail or populate the related fields only when getTelemetrySavedObject succeeds.

Guida contributor

Direzione di ricerca
Esamina il codice del collector del plugin di telemetria, in particolare il blocco try catch che sopprime gli errori da getTelemetrySavedObject. Determina se è più sicuro far fallire il collector in caso di errore o popolare i campi solo in caso di successo. Considera l'impatto sulla segnalazione dei dati quando la connessione ES fallisce.
Tech stack
typescript
Dominio
backend
Tipo issue
Bug
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.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70