Skip to content

[box_events] Migrate to the CEL input - #20879

Open
chrisberkhout wants to merge 8 commits into
elastic:mainfrom
chrisberkhout:box_events-cel-migration
Open

[box_events] Migrate to the CEL input#20879
chrisberkhout wants to merge 8 commits into
elastic:mainfrom
chrisberkhout:box_events-cel-migration

Conversation

@chrisberkhout

@chrisberkhout chrisberkhout commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Proposed commit message

[box_events] Migrate to the CEL input

Migrate from the HTTPJSON input to the CEL input, parsing responses so
that the `next_stream_position` cursor value is handled as a string
end-to-end. This fixes rounding of stream position values above 2^53,
which could cause data gaps.

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

@chrisberkhout chrisberkhout self-assigned this Aug 24, 2026
@chrisberkhout
chrisberkhout requested review from a team as code owners August 24, 2026 14:28
@chrisberkhout chrisberkhout added Integration:box_events Box Events bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Aug 24, 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.

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

Comment thread packages/box_events/data_stream/events/agent/stream/cel.yml.hbs Outdated
Comment thread packages/box_events/data_stream/events/agent/stream/cel.yml.hbs Outdated
Comment thread packages/box_events/data_stream/events/agent/stream/cel.yml.hbs Outdated
Comment thread packages/box_events/data_stream/events/agent/stream/cel.yml.hbs Outdated
Comment thread packages/box_events/data_stream/events/agent/stream/cel.yml.hbs Outdated
@chrisberkhout

Copy link
Copy Markdown
Contributor Author

@efd6 Thanks for the CEL edits. I did add a sentence about admin_logs_streaming for admin_logs.

@chrisberkhout
chrisberkhout requested a review from efd6 August 25, 2026 08:33
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

✅ All changelog entries have the correct PR link.

@vera-review-bot

Copy link
Copy Markdown

🟢 No issues across the latest commits 4be0916, efbbff2.

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 - v0.2.7 | 📚 Knowledge base: integration-skills

⚠️ Automated review — verify suggestions before applying.

@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @chrisberkhout

@efd6 efd6 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.

I'm happy with this but let's wait for PM review.

@mergify

mergify Bot commented Aug 26, 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

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:box_events Box Events 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.

box_events: cursor offset is outside range of exact integer representation

2 participants