-
Notifications
You must be signed in to change notification settings - Fork 387
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] [compute_vm_scaleset] List exposed metrics; add dimension and metric_type metadata #7177
[Azure] [compute_vm_scaleset] List exposed metrics; add dimension and metric_type metadata #7177
Conversation
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
…amespace; add virtual_machine dimension; add changelog record Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
🌐 Coverage report
|
type: float | ||
metric_type: gauge | ||
description: Bytes/Sec read from a single disk during monitoring period for OS disk | ||
- name: memory_commit_limit.avg |
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.
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 changes look good.
Did you get a chance to collect the missing metric in your tests? I'm not an expert here, but I'm available if you wan to try to get it before merging.
no, I wasn't able to get this one metric Also I wasn't able to find the full list of metrics, that are supposed to be available in the azure documentation, maybe you know about such documentation? |
- name: dimensions | ||
type: group | ||
fields: | ||
- name: "vmname" |
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.
Referring to https://learn.microsoft.com/en-us/azure/azure-monitor/reference/supported-metrics/microsoft-compute-virtualmachinescalesets-metrics, which field holds LUN value?
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.
there is no such field exposed, list of dimensions is defined in configuration - https://github.com/elastic/integrations/blob/main/packages/azure_metrics/data_stream/compute_vm_scaleset/agent/stream/stream.yml.hbs#L71-L73
from my understanding - in this case metric should be grouped by vmname
only, if only this dimension is defined
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.
I've added lun
azure dimension in this PR - #7314
coult you please have another look on this PR?
I can't find this metric either. I suppose you already checked at https://learn.microsoft.com/en-us/azure/azure-monitor/reference/supported-metrics/metrics-index; I remember this page in your browser's tabs. |
yes, I've seen it, but it does not contain the guest VM metrics there |
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
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!
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
Package azure_metrics - 1.0.23 containing this change is available at https://epr.elastic.co/search?package=azure_metrics |
… metric_type metadata (#7177) * add dimensions and metric_type for available metrics Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co> * add metrics, that are coming from the Azure.VM.Windows.GuestMetrics namespace; add virtual_machine dimension; add changelog record Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co> * fix PR link * update doxumentation with the new dimension field Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co> --------- Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
What does this PR do?
vmname
) - https://learn.microsoft.com/en-us/azure/azure-monitor/reference/supported-metrics/microsoft-compute-virtualmachinescalesets-metricscloud.region
azure.resource.id
- this field includesazure.subscription_id
,azure.resource.group
,azure.resource.name
, format:azure.dimensions.vmname
azure.dimensions.virtual_machine
azure.timegrain
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots