Use EBS* instead of Disk* metrics in EC2 dashboards and alerts - #19794
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the AWS EC2 OTel package’s Kibana assets to use EC2 EBS* metrics (instead of Disk*) for storage-related dashboards and alerting, aligning the integration with Nitro/EBS-backed instance behavior.
Changes:
- Updated the EC2 dashboard ESQL queries and labels from
Disk*toEBS*metrics (bytes/ops) and renamed related panel/layer IDs. - Added new alerting rule templates for high EBS read/write throughput using
EBSReadBytes/EBSWriteBytes. - Removed the previous local disk throughput alert templates.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/aws_ec2_otel/kibana/dashboard/aws_ec2_otel-45f45575-d519-4a70-a50c-2195c0c9f916.json | Replaces Disk* metric usage with EBS* in dashboard queries/labels and updates panel identifiers. |
| packages/aws_ec2_otel/kibana/alerting_rule_template/aws_ec2_otel-high-ebs-write-throughput.json | Adds managed alert template for high EBS write throughput (EBSWriteBytes). |
| packages/aws_ec2_otel/kibana/alerting_rule_template/aws_ec2_otel-high-ebs-read-throughput.json | Adds managed alert template for high EBS read throughput (EBSReadBytes). |
| packages/aws_ec2_otel/kibana/alerting_rule_template/aws_ec2_otel-high-disk-write-throughput.json | Removes the previous local disk write throughput alert template. |
| packages/aws_ec2_otel/kibana/alerting_rule_template/aws_ec2_otel-high-disk-read-throughput.json | Removes the previous local disk read throughput alert template. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
💚 Build Succeeded
|
|
@tommyers-elastic please update README since alerts changed You can generate with |
|
@jakubgalecki0 done. but how come the pre-rendered README wasn't committed before? |
✅ Elastic Docs Style Checker (Vale)No issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
@tommyers-elastic it's my bad I thought I did commit them. I will create PR with docs for remaining packages soon |
|
✅ All changelog entries have the correct PR link. |
💚 Build Succeeded
History
|
|
Package aws_ec2_otel - 0.6.0 containing this change is available at https://epr.elastic.co/package/aws_ec2_otel/0.6.0/ |
Proposed commit message
Most if not all general purpose instance types use nitro/EBS instead of physical disks. we swap existing usages of
Disk*metrics e.g.DiskWriteOpswith the equivalentEBS*metric from the EC2 namespace e.g.EBSWriteOps.Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots