Skip to content

[Azure Storage Account] Add default_timegrain configuration option - #15501

Merged
zmoog merged 3 commits into
mainfrom
zmoog/feat/azure/storage/default-timegrain
Oct 6, 2025
Merged

[Azure Storage Account] Add default_timegrain configuration option#15501
zmoog merged 3 commits into
mainfrom
zmoog/feat/azure/storage/default-timegrain

Conversation

@zmoog

@zmoog zmoog commented Sep 30, 2025

Copy link
Copy Markdown
Contributor

Proposed commit message

Adds a new "Default Timegrain" configuration option to allow users to customize the timegrain used in the Storage Account integration. The default value remains PT5M, but users can now choose a different value.

Without this option, users can only collect metrics with a PT5M time grain. It is a sensible default, but some users want to collect metrics with a PT1M time grain.

To learn more, see #15464.

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

Related issues

Screenshots

CleanShot 2025-10-06 at 11 21 00@2x
@zmoog zmoog self-assigned this Sep 30, 2025
@zmoog zmoog added Integration:azure_metrics Azure Resource Metrics Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] labels Sep 30, 2025
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@zmoog
zmoog marked this pull request as ready for review October 6, 2025 09:21
@zmoog
zmoog requested a review from a team as a code owner October 6, 2025 09:21
zmoog and others added 3 commits October 6, 2025 11:29
# Conflicts:
#	packages/azure_metrics/changelog.yml
#	packages/azure_metrics/manifest.yml
@zmoog
zmoog force-pushed the zmoog/feat/azure/storage/default-timegrain branch from 3f183f0 to 29574ea Compare October 6, 2025 09:30
@zmoog

zmoog commented Oct 6, 2025

Copy link
Copy Markdown
Contributor Author

I will run a final test when 8.18.8, 8.19.5, 9.0.8, and 9.1.5 will be available.

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @zmoog

@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Oct 6, 2025
@zmoog

zmoog commented Oct 6, 2025

Copy link
Copy Markdown
Contributor Author

Quick test on 8.18.8 with PT1M/300s:

CleanShot 2025-10-06 at 22 28 02@2x

I could collect PT1M metrics:

CleanShot 2025-10-06 at 22 26 01@2x
@zmoog
zmoog merged commit 1bf663f into main Oct 6, 2025
7 checks passed
@zmoog
zmoog deleted the zmoog/feat/azure/storage/default-timegrain branch October 6, 2025 20:30
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

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

agithomas pushed a commit to agithomas/integrations that referenced this pull request Oct 30, 2025
…lastic#15501)

Adds a new "Default Timegrain" configuration option to allow users to customize the timegrain used in the Storage Account integration. The default value remains PT5M, but users can now choose a different value.

Without this option, users can only collect metrics with a PT5M time grain. It is a sensible default, but some users want to collect metrics with a PT1M time grain.

To learn more, see elastic#15464.
tehbooom pushed a commit to tehbooom/integrations that referenced this pull request Nov 19, 2025
…lastic#15501)

Adds a new "Default Timegrain" configuration option to allow users to customize the timegrain used in the Storage Account integration. The default value remains PT5M, but users can now choose a different value.

Without this option, users can only collect metrics with a PT5M time grain. It is a sensible default, but some users want to collect metrics with a PT1M time grain.

To learn more, see elastic#15464.
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]

5 participants