elastic / kibana

Your window into the Elastic Stack
https://www.elastic.co/products/kibana
Other
19.57k stars 8.09k forks source link

[Stack Monitoring] APM Server instance page throws error if summary is missing #154137

Open miltonhultgren opened 1 year ago

miltonhultgren commented 1 year ago

If the collection side is facing issues with gathering the Beat stats for the APM/Integration server then the two calls made here to fetch the stats and the timestamp of the latest ingestion both turn up undefined (so {} is returned thanks to io-ts stripping away any undefined keys).

This results in this code here to throw since config is undefined (even if it's available in the global plugin config).

We could simply guard that with an optional chaining access or read from the plugin config in the browser, but I feel like it would make more sense to expose that this data is missing to the user since it likely indicates an error in the collection side.

AC

elasticmachine commented 1 year ago

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)