search

elastic / kibana

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

[Infra UI] AWS EC2 instances not showing up in the Inventory UI #182988

Closed neptunian closed 3 months ago

neptunian commented 6 months ago

The query when using the dropdown the Inventory UI waffle map is looking for term: { 'event.dataset': 'aws.ec2' } instead of term: { 'event.dataset': 'aws.ec2_metrics' }

elasticmachine commented 6 months ago

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

roshan-elastic commented 6 months ago

Thanks for this @neptunian.

Quick question on this...(and please defer this if you shouldn't be answering this right now - I can ask @smith in our weekly backlog catch-up)

If we deliver this issue, do you know what users will see when they select 'EC2' in inventory from the drop-down?

Specifically:

I have a feeling that as we deprecated the 'AWS' section (as we saw little to no usage) of hosts...we might need to do more than just fix the query.

smith commented 6 months ago

If we deliver this issue, do you know what users will see when they select 'EC2' in inventory from the drop-down?

Specifically:

Will they see 'squares' for each one?

Yes.

Will the metrics show up for each host (in each square)?

Yes. It's not a host it's an EC2 instance. (According to the current inventory model. The metrics are different.)

What will happen if they click on a square to see more about the EC2?

Similar to what you see for Kubernetes pods today, a dropdown with links to filtered logs, metrics, and APM views, filtered by cloud.instance.id. I don't have a running environment with these but am inferring it from the inventory model: https://github.com/elastic/kibana/blob/main/x-pack/plugins/observability_solution/metrics_data_access/common/inventory_models/aws_ec2/index.ts#L26-L37.

I have a feeling that as we deprecated the 'AWS' section (as we saw little to no usage) of hosts...we might need to do more than just fix the query.

The entity discovery framework should be able to discover host and/or cloud.instance entities from CloudWatch log and metric data. You'll have to talk to the team about that.

Fixing this specific bug requires deleting eight characters so we should do so once we get a chance.

roshan-elastic commented 3 months ago

Closing this as we'll handle this in a new inventory