Skip to content

Conversation

@zmoog
Copy link
Contributor

@zmoog zmoog commented May 27, 2025

Proposed commit message

Add the Latency config option to Azure Monitor integrations.

Latency is the time it takes for the Azure service to publish the metric values to Azure Monitor. The integration uses the latency value to compensate for the delay in metric value publishing.

Default value is 0. Typical values are 30s or 1m.

We expect the latency to help integrations with a few services that report metrics with delay.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • Check the latency on 8.17.7

Related issues

Screenshots

CleanShot 2025-05-27 at 19 13 23@2x

CleanShot 2025-05-27 at 19 14 37@2x

@andrewkroh andrewkroh added the Integration:azure_metrics Azure Resource Metrics label May 27, 2025
@zmoog zmoog self-assigned this May 27, 2025
@zmoog zmoog added the Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] label May 27, 2025
@zmoog zmoog marked this pull request as ready for review May 27, 2025 17:20
@zmoog zmoog requested a review from a team as a code owner May 27, 2025 17:20
Copy link
Member

@vigneshshanmugam vigneshshanmugam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MichaelKatsoulis
Copy link
Contributor

Let's merge #13783 first as it includes changes needed also for 9.0.1

@zmoog zmoog force-pushed the zmoog/feat/azure-metrics-latency branch from 1c7e175 to 6abd5ce Compare June 3, 2025 08:35
@elastic-vault-github-plugin-prod

🚀 Benchmarks report

Package azure_metrics 👍(0) 💚(2) 💔(2)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
compute_vm 90909.09 76923.08 -13986.01 (-15.38%) 💔
container_instance 333333.33 166666.67 -166666.66 (-50%) 💔

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @zmoog

@zmoog zmoog merged commit 7785eb1 into elastic:main Jun 3, 2025
8 checks passed
@zmoog zmoog deleted the zmoog/feat/azure-metrics-latency branch June 3, 2025 10:36
@elastic-vault-github-plugin-prod

Package azure_metrics - 1.9.0 containing this change is available at https://epr.elastic.co/package/azure_metrics/1.9.0/

@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:azure_metrics Azure Resource Metrics Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services]

7 participants