Skip to content
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 monitor] Metric values are not mapped to the correct metric definition #42536

Open
MichaelKatsoulis opened this issue Jan 31, 2025 · 2 comments
Assignees
Labels
bug Team:obs-ds-hosted-services Label for the Observability Hosted Services team

Comments

@MichaelKatsoulis
Copy link
Contributor

In Azure monitor metricset, after the metric values are collected from Azure api, the metricdefinition for which the values got fetched, is trying to be matched against the metric definitions stored in a in memory store.

if matchMetrics(currentMetric, metric) {

For the matching process, Aggregations, Names , ResourceId and prevMet.TimeGrain is compared.

if prevMet.Namespace == met.Namespace && reflect.DeepEqual(prevMet.Names, met.Names) && prevMet.ResourceId == met.ResourceId &&

But there are cases that the metric definitions have all these fields the same but differ in Dimensions field.
That leads to the following:

@MichaelKatsoulis MichaelKatsoulis self-assigned this Jan 31, 2025
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 31, 2025
@MichaelKatsoulis MichaelKatsoulis added Team:obs-ds-hosted-services Label for the Observability Hosted Services team and removed needs_team Indicates that the issue/PR needs a Team:* label labels Jan 31, 2025
@elasticmachine
Copy link
Collaborator

Pinging @elastic/obs-ds-hosted-services (Team:obs-ds-hosted-services)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Team:obs-ds-hosted-services Label for the Observability Hosted Services team
Projects
None yet
Development

No branches or pull requests

2 participants