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 metricset only collect one aggregation type at a time #39192

Closed
zmoog opened this issue Apr 24, 2024 · 3 comments · Fixed by #39204
Closed

Azure Monitor metricset only collect one aggregation type at a time #39192

zmoog opened this issue Apr 24, 2024 · 3 comments · Fixed by #39204
Assignees
Labels
bug Team:obs-ds-hosted-services Label for the Observability Hosted Services team

Comments

@zmoog
Copy link
Contributor

zmoog commented Apr 24, 2024

  • Version: 8.13.2
  • Operating System: any
  • Steps to Reproduce:

A user reported the Azure Monitor metricset is collecting only one aggregation type despite setting up a configuration for three (Maximum, Minimum, Avergate).

The following two configuration variants for Microsoft.EventHub/Namespaces only return one metric instead of the expected three.

- module: azure
  metricsets:
    - monitor
  enabled: true
  period: 60s
  client_id: '<redacted>'
  client_secret: '<redacted>'
  tenant_id: '<redacted>'
  subscription_id: '<redacted>'
  refresh_list_interval: 600s
  resources:
  - metrics:
    - aggregations:
        - Maximum
      ignore_unsupported: true
      name:
        - ActiveConnections
      namespace: Microsoft.EventHub/Namespaces
    - aggregations:
        - Minimum
      ignore_unsupported: true
      name:
        - ActiveConnections
      namespace: Microsoft.EventHub/Namespaces
    - aggregations:
        - Average
      ignore_unsupported: true
      name:
        - ActiveConnections
      namespace: Microsoft.EventHub/Namespaces
- module: azure
  metricsets:
    - monitor
  enabled: true
  period: 60s
  client_id: '<redacted>'
  client_secret: '<redacted>'
  tenant_id: '<redacted>'
  subscription_id: '<redacted>'
  refresh_list_interval: 600s
  resources:
  - metrics:
      namespace: Microsoft.EventHub/Namespaces
      aggregations:
        - Average
        - Maximum
        - Minimum
      ignore_unsupported: true
      timegrain: PT1M
@zmoog zmoog added bug Team:obs-ds-hosted-services Label for the Observability Hosted Services team labels Apr 24, 2024
@zmoog zmoog self-assigned this Apr 24, 2024
@elasticmachine
Copy link
Collaborator

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

@zmoog
Copy link
Contributor Author

zmoog commented Apr 24, 2024

I can confirm this is a regression bug introduced in 8.12.

I'm adding the missing tests to ensure the multiple aggregations scenario becomes part of the test suite and we never miss this again.

@zmoog
Copy link
Contributor Author

zmoog commented Apr 29, 2024

The fix will be available in the following versions:

  • 8.15.0
  • 8.14.0

I am also backporting it to 8.14.0 and attempting to include it in 8.13.3, but we may not make it in time.

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

Successfully merging a pull request may close this issue.

2 participants