[telemetry] Clean up cloud provider usage collector & add retry logic. #96711
Labels
enhancement
New value added to drive a business result
Feature:Telemetry
Team:Core
Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc
In #95131, we migrated some legacy "cloud detector" logic from the
monitoring
plugin intokibana_usage_collection
in order to collect some basic information about what cloud environment Kibana is running in via a newcloud_provider
usage collector. (src/plugins/kibana_usage_collection/server/collectors/cloud
)In reviewing the old code, we identified a few enhancements & cleanup tasks we'd like to address, some of which we didn't do at the time as it would have required some more heavy refactoring:
request
,fs
), and mocking those in the testsrequest
lib in favor of something likenode-fetch
which we use more widely in kibana (request looks like it's mostly used in tests atm, not many other places)[0] Some of the API docs have info on possible error codes that can be returned, which we should take a look at when determining our retry logic:
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/instance-metadata-service
https://cloud.google.com/compute/docs/storing-retrieving-metadata
https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/instancedata-data-retrieval.html
The text was updated successfully, but these errors were encountered: