-
Notifications
You must be signed in to change notification settings - Fork 476
[Azure][container_registry][compute_vm_scaleset] add missing dimensions to the container_registry and compute_vm_scaleset #7314
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Azure][container_registry][compute_vm_scaleset] add missing dimensions to the container_registry and compute_vm_scaleset #7314
Conversation
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
…link in changelog Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
🌐 Coverage report
|
packages/azure_metrics/data_stream/container_registry/agent/stream/stream.yml.hbs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change looks good, maybe we're only missing a couple of dimensions
attributes in the Agent configuration template.
packages/azure_metrics/data_stream/container_registry/agent/stream/stream.yml.hbs
Outdated
Show resolved
Hide resolved
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
@zmoog could you please have another look on this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Package azure_metrics - 1.0.20 containing this change is available at https://epr.elastic.co/search?package=azure_metrics |
What does this PR do?
container_registry
:add geolocation as a dimension - https://learn.microsoft.com/en-us/azure/azure-monitor/reference/supported-metrics/microsoft-containerregistry-registries-metrics
use case for this: https://learn.microsoft.com/en-us/azure/container-registry/container-registry-geo-replication
compute_vm_scaleset
some metrics additionally to the
VMname
haveLUN
(Logical Unit Number (LUN) is a number that is used to identify a specific storage device) as a dimensionChecklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots