Skip to content

[lumos] Fix activity logs cursor so incremental collection filters requests by the stored timestamp - #20279

Merged
brijesh-elastic merged 3 commits into
elastic:mainfrom
brijesh-elastic:fix-lumos-activity-logs-cursor-18494
Jul 24, 2026
Merged

[lumos] Fix activity logs cursor so incremental collection filters requests by the stored timestamp#20279
brijesh-elastic merged 3 commits into
elastic:mainfrom
brijesh-elastic:fix-lumos-activity-logs-cursor-18494

Conversation

@brijesh-elastic

Copy link
Copy Markdown
Contributor

Proposed commit message

lumos: Fix activity logs cursor so incremental collection filters requests by the stored timestamp
instead of replaying the full history each interval

Lumos Activity API details are (here)[1].

[1] https://developers.lumos.com/reference/getactivitylogs-1

Note

To Reviewers:

  • This changes has not been tested with a live instance.

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

How to test this PR locally

  • Clone integrations repo.
  • Install elastic package locally.
  • Start elastic stack using elastic-package.
  • Move to integrations/packages/lumos directory.
  • Run the following command to run tests.

elastic-package test -v

Related issues

@brijesh-elastic brijesh-elastic self-assigned this Jul 22, 2026
@brijesh-elastic
brijesh-elastic requested review from a team as code owners July 22, 2026 09:46
@brijesh-elastic brijesh-elastic added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Integration:lumos Lumos (Partner supported) Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] labels Jul 22, 2026
@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown
Contributor

✅ Elastic Docs Style Checker (Vale)

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@vera-review-bot

Copy link
Copy Markdown

No issues across the latest commits 2979d68.

A new commit triggers another review — at most once every 15 minutes. I skip the PR while it's approved or has merge conflicts.

🤖 AI-Generated Review | Vera Review Bot | 📚 Knowledge base: integration-skills

⚠️ Automated review — verify suggestions before applying.

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@ShourieG ShourieG left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mergify

mergify Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

"given_name": "Coyote"
"email": "wile.e.coyote@example.com",
"family_name": "Coyote",
"given_name": "Wile"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"given_name": "Wile"
"given_name": "Wile E."

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

✅ All changelog entries have the correct PR link.

@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @brijesh-elastic

@brijesh-elastic
brijesh-elastic merged commit 7cd9564 into elastic:main Jul 24, 2026
10 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

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

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 documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:lumos Lumos (Partner supported) Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] 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.

[httpjson-pagination] Lumos activity_logs cursor is never applied, causing full replay each interval

3 participants