elastic/kibana

[Telemetry] Better error handling at the `telemetry_plugin_collector`

Open

#122,634 opened on 2022年1月11日

GitHub で見る
 (4 comments) (1 reaction) (0 assignees)TypeScript (19,065 stars) (8,021 forks)batch import
Feature:TelemetryTeam:Coregood first issueimpact:lowloe:small

説明

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.

コントリビューターガイド

技術スタック
typescriptnodejs
領域
backendobservability
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
TypeScriptNode.jsElasticsearchKibana telemetry
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
50
調査方針
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.