Skip to content

Conversation

@chrisberkhout
Copy link
Contributor

Proposed commit message

[o365] No delays until up-to-date

Instead of fetching history continuously until reaching the current day
then waiting the interval time between requests, keep fetching until
within one batch length of the current time.

With the default settings, waiting 3 minutes between requests for one
hour batches would significantly extend the time required to get
up-to-date. For example, it's 45 minutes of waiting to cover 15 hours.
It's particularly disruptive to the user experience if there is no
historical data and new data has such a 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

@chrisberkhout chrisberkhout self-assigned this Jul 21, 2025
@chrisberkhout chrisberkhout requested a review from a team as a code owner July 21, 2025 13:16
@chrisberkhout chrisberkhout added Integration:o365 Microsoft Office 365 bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Jul 21, 2025
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@chrisberkhout chrisberkhout requested a review from efd6 July 22, 2025 08:46
@elastic-sonarqube
Copy link

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @chrisberkhout

Copy link
Contributor

@efd6 efd6 left a comment

Choose a reason for hiding this comment

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

Thanks

@chrisberkhout chrisberkhout merged commit b17cd09 into elastic:main Jul 23, 2025
9 checks passed
@elastic-vault-github-plugin-prod

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes a bug issue Integration:o365 Microsoft Office 365 Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants