elastic/kibana

[Telemetry] Better error handling at the `telemetry_plugin_collector`

Open

#122.634 geöffnet am 11. Jan. 2022

Auf GitHub ansehen
 (4 Kommentare) (1 Reaktion) (0 zugewiesene Personen)TypeScript (19.065 Stars) (8.021 Forks)batch import
Feature:TelemetryTeam:Coregood first issueimpact:lowloe:small

Beschreibung

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.

Contributor Guide

Tech Stack
typescriptnodejs
Domain
backendobservability
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
TypeScriptNode.jsElasticsearchKibana telemetry
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50
Research-Richtung
The issue is located in `src/plugins/telemetry/server/collectors/telemetry plugin/telemetry plugin collector.ts` around line 45-52, where errors from `getTelemetrySavedObject` are currently swallowed. The two suggested approaches are to either let the collector fail or only populate fields when the call succeeds. A contributor should evaluate which approach is more appropriate for consistent telemetry data and implement the change. No linked PRs or assignees exist, so this is an open opportunity.